Alban Gruin <alban.gruin@xxxxxxxxx> writes: > + ret = fputs(buf.buf, todo); > + if (ret < 0) > + error_errno(_("Could not append help text to '%s'"), rebase_path_todo()); Error checking fputs() return is an improvement from the version that has been queued on 'pu'. I think messages given to error() and friends begin with lowercase uncapitalized by convention, though. > + fclose(todo); > + strbuf_release(&buf); > + > + return ret; > +} > diff --git a/rebase-interactive.h b/rebase-interactive.h > new file mode 100644 > index 000000000..47372624e > --- /dev/null > +++ b/rebase-interactive.h > @@ -0,0 +1,6 @@ > +#ifndef REBASE_INTERACTIVE_H > +#define REBASE_INTERACTIVE_H > + > +int append_todo_help(unsigned edit_todo, unsigned keep_empty); > + > +#endif