[patch] Add -q in help info and doc

[patch] Add -q in help info and doc

[Mingjie Xing]

[-- Attachment #1: Type: text/plain, Size: 384 bytes --]

Hello,

I notice that "-q" and "--silent" options are not mentioned in gdb's
help information. This patch just adds them and updates the document.

Besides, most options in gdb document are using "-" prefix instead of "--".

ChangeLog

        * gdb.texinfo (Invoking GDB): Add -q and --quiet.
        * main.c (print_gdb_help): Add -q and --silent.

Is it OK?

Best regards,
Mingjie

[-- Attachment #2: gdb-help-info.patch --]
[-- Type: text/x-patch, Size: 1154 bytes --]

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 9f7fa18..a0fb66d 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -885,10 +885,11 @@ This will cause @code{@value{GDBP}} to debug @code{gcc}, and to set
 @code{gcc}'s command-line arguments (@pxref{Arguments}) to @samp{-O2 -c foo.c}.
 
 You can run @code{@value{GDBP}} without printing the front material, which describes
-@value{GDBN}'s non-warranty, by specifying @code{-silent}:
+@value{GDBN}'s non-warranty, by specifying @code{--silent}
+(or @code{-q}/@code{--quiet}):
 
 @smallexample
-@value{GDBP} -silent
+@value{GDBP} --silent
 @end smallexample
 
 @noindent
diff --git a/gdb/main.c b/gdb/main.c
index a9fc378..108759d 100644
--- a/gdb/main.c
+++ b/gdb/main.c
@@ -1173,7 +1173,8 @@ Output and user interface control:\n\n\
   fputs_unfiltered (_("\
   --dbx              DBX compatibility mode.\n\
   --xdb              XDB compatibility mode.\n\
-  --quiet            Do not print version number on startup.\n\n\
+  -q, --quiet, --silent\n\
+                     Do not print version number on startup.\n\n\
 "), stream);
   fputs_unfiltered (_("\
 Operating modes:\n\n\

Re: [patch] Add -q in help info and doc

[Eli Zaretskii]

> From: Mingjie Xing <mingjie.xing@gmail.com>
> Date: Fri, 6 Jun 2014 16:12:35 +0800
> 
> I notice that "-q" and "--silent" options are not mentioned in gdb's
> help information. This patch just adds them and updates the document.
> 
> Besides, most options in gdb document are using "-" prefix instead of "--".
> 
> ChangeLog
> 
>         * gdb.texinfo (Invoking GDB): Add -q and --quiet.
>         * main.c (print_gdb_help): Add -q and --silent.
> 
> Is it OK?

Fine with me, thanks.

Re: [patch] Add -q in help info and doc

[Pedro Alves]

On 06/06/2014 09:12 AM, Mingjie Xing wrote:
> Hello,
> 
> I notice that "-q" and "--silent" options are not mentioned in gdb's
> help information. This patch just adds them and updates the document.

I notice it's missing from the man page too.

(cd doc; make man; man ./gdb.1)

-- 
Pedro Alves

Re: [patch] Add -q in help info and doc

[Mingjie Xing]

2014-06-06 18:26 GMT+08:00 Pedro Alves <palves@redhat.com>:
> I notice it's missing from the man page too.
>
> (cd doc; make man; man ./gdb.1)

Yes. But before fixing it, I think It should be better that we can
firstly unify the
long option format. Currently, we use "--" prefix in the help
information, while "-"
in gdb.texinfo. It's confusing for users, though gdb accepts both kinds.

Best regards,
Mingjie

Re: [patch] Add -q in help info and doc

[Sergio Durigan Junior]

On Sunday, June 08 2014, Mingjie Xing wrote:

> 2014-06-06 18:26 GMT+08:00 Pedro Alves <palves@redhat.com>:
>> I notice it's missing from the man page too.
>>
>> (cd doc; make man; man ./gdb.1)
>
> Yes. But before fixing it, I think It should be better that we can
> firstly unify the
> long option format. Currently, we use "--" prefix in the help
> information, while "-"
> in gdb.texinfo. It's confusing for users, though gdb accepts both kinds.

I agree.  And last time I checked, I saw that our manpage is missing
useful options.  I was planning to tackle this in some spare time (not
likely anytime soon), but if Mingjie wants to do that too...

Thanks,

-- 
Sergio
GPG key ID: 65FC5E36
Please send encrypted e-mail if possible
http://blog.sergiodj.net/

Re: [patch] Add -q in help info and doc

[Pedro Alves]

On 06/09/2014 02:32 AM, Mingjie Xing wrote:
> 2014-06-06 18:26 GMT+08:00 Pedro Alves <palves@redhat.com>:
>> I notice it's missing from the man page too.
>>
>> (cd doc; make man; man ./gdb.1)
> 
> Yes. But before fixing it, I think It should be better that we can
> firstly unify the
> long option format. Currently, we use "--" prefix in the help
> information, while "-"
> in gdb.texinfo. It's confusing for users, though gdb accepts both kinds.

Yes, it's confusing, agreed, but I'd call that an orthogonal issue.

But if you want to fix both issues, then that's great.

TBC, I'm not suggesting you _have_ to fix the man page.

-- 
Pedro Alves

Re: [patch] Add -q in help info and doc

[Mingjie Xing]

2014-06-09 17:33 GMT+08:00 Pedro Alves <palves@redhat.com>:
>
> TBC, I'm not suggesting you _have_ to fix the man page.

Thanks. I've sent a patch to share options between man page and info
page, which should fix it.

Best regards,
Mingjie