Re: [PATCH v5 14/16] merge-recursive: offer an option to retain the output in 'obuf'

> Johannes Schindelin <johannes.schindelin@gmx.de> writes:
> 
> > diff --git a/merge-recursive.h b/merge-recursive.h
> > index d415724..340704c 100644
> > --- a/merge-recursive.h
> > +++ b/merge-recursive.h
> > @@ -13,7 +13,7 @@ struct merge_options {
> >  		MERGE_RECURSIVE_THEIRS
> >  	} recursive_variant;
> >  	const char *subtree_shift;
> > -	unsigned buffer_output : 1;
> > +	unsigned buffer_output : 2; /* 1: output at end, 2: keep buffered */
> >  	unsigned renormalize : 1;
> 
> Once a field ceases to be a boolean, it is OK not to squish it into
> a bitfield like this for a struct that we will have only a very
> small number of instances of.  Treating it just like "verbosity",
> which occupies a whole int even though it can only get up to 5 or
> so, would be more appropriate.