Re: [PATCH] Remove --no-gui option from difftool usage string

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 03, 2017 at 09:58:09PM -0800, Jacob Keller wrote:
> On Fri, Feb 3, 2017 at 6:56 PM, Denton Liu <liu.denton@xxxxxxxxx> wrote:
> > The --no-gui option not documented in the manpage, nor is it actually
> > used in the source code. This change removes it from the usage help
> > that's printed.
> >
> > Signed-off-by: Denton Liu <liu.denton@xxxxxxxxx>
> > ---
> >  git-difftool.perl | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/git-difftool.perl b/git-difftool.perl
> > index a5790d03a..657d5622d 100755
> > --- a/git-difftool.perl
> > +++ b/git-difftool.perl
> > @@ -29,8 +29,8 @@ sub usage
> >         print << 'USAGE';
> >  usage: git difftool [-t|--tool=<tool>] [--tool-help]
> >                      [-x|--extcmd=<cmd>]
> > -                    [-g|--gui] [--no-gui]
> > -                    [--prompt] [-y|--no-prompt]
> > +                    [-g|--gui] [--prompt]
> > +                    [-y|--no-prompt]
> >                      [-d|--dir-diff]
> >                      ['git diff' options]
> >  USAGE
> > --
> > 2.11.0
> >
> 
> Aren't "--no-foo" options automatically created for booleans when
> using our option parsing code?
> 
> Thanks,
> Jake

Sorry, I guess I didn't notice that. Would it make sense to document it
in the manpage then?

--
Denton Liu



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]