On Tue, 2 Mar 2021 at 04:06, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote:
>
> > + if (argc)
> > + die(_("cannot combine reword option of --fixup with path %s"), *argv);
> > + if (patch_interactive)
> > + die(_("cannot combine reword option of --fixup with --patch"));
> > + if (interactive)
> > + die(_("cannot combine reword option of --fixup with --interactive"));
> > + if (all)
> > + die(_("cannot combine reword option of --fixup with --all"));
> > + if (also)
> > + die(_("cannot combine reword option of --fixup with --include"));
> > + if (only)
> > + die(_("cannot combine reword option of --fixup with --only"));
> > +}
>
> Or, more concisely:
>
> if (argc)
> die(_("--fixup mutually exclusive with path '%s'), ...);
> if (patch_interactive || interactive || all || also || only)
> die(_("--fixup mutually exclusive with
> --patch/--interactive/--all/--include/--only);
>
Got it ! Its seems more clear, I will fix it.
> The mix of two different error message styles (capitalized with
> full-stop vs. lowercase no-full-stop) is a bit jarring, but minor.
>
Okay, I will fix it.
