Re: [PATCH 2/3] sequencer: make commit options more extensible
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Mar 23, 2017, 14:39 UTC
- Message-ID
- <alpine.DEB.2.20.1703231539010.3767@virtualbox>
- In-Reply-To
- <xmqqa88dkw9j.fsf@gitster.mtv.corp.google.com>
Hi Junio,
On Wed, 22 Mar 2017, Junio C Hamano wrote:
Show 12 quoted lines
> Johannes Schindelin <johannes.schindelin@gmx.de> writes:
>
> > @@ -615,8 +620,7 @@ N_("you have staged changes in your working tree\n"
> > * author metadata.
> > */
> > static int run_git_commit(const char *defmsg, struct replay_opts *opts,
> > - int allow_empty, int edit, int amend,
> > - int cleanup_commit_message)
> > + int flags)
>
> Use "unsigned" not signed integer for a collection of bits (see
> e.g. GSoC microproject ideas page).True. Fixed in preparation for v2.
Ciao, Dscho