Re: [PATCH v6 1/9] pack-objects: use standard option incompatibility functions

> Taylor Blau <me@ttaylorr.com> writes:
>
> > @@ -5050,13 +5051,14 @@ int cmd_pack_objects(int argc,
> >  	if (!pack_to_stdout && thin)
> >  		die(_("--thin cannot be used to build an indexable pack"));
> >
> > -	if (keep_unreachable && unpack_unreachable)
> > -		die(_("options '%s' and '%s' cannot be used together"), "--keep-unreachable", "--unpack-unreachable");
> > +	die_for_incompatible_opt2(keep_unreachable, "--keep-unreachable",
> > +				  unpack_unreachable, "--unpack-unreachable");
> >  	if (!rev_list_all || !rev_list_reflog || !rev_list_index)
> >  		unpack_unreachable_expiration = 0;
> >
> > -	if (stdin_packs && filter_options.choice)
> > -		die(_("cannot use --filter with --stdin-packs"));
> > +	die_for_incompatible_opt2(stdin_packs, "--stdin-packs",
> > +				  filter_options.choice, "--filter");
> > +
> >
>
> We do not need two blank lines here, do we?