This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Improve "help all"
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Vladimir Prus <vladimir at codesourcery dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Sat, 28 Oct 2006 13:28:57 +0200
- Subject: Re: Improve "help all"
- References: <200610272340.26372.vladimir@codesourcery.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> From: Vladimir Prus <vladimir@codesourcery.com>
> Date: Fri, 27 Oct 2006 23:40:26 +0400
>
> at the moment gdb's "help all" command is not as helpful as it could be
> because:
>
> 1. It's not mentioned in the output of "help".
> 2. The output of "help all" lacks any structure -- like visual grouping by
> classes.
> 3. The prefix commands are printed in weird order. For example, the
> "append" and "append binary" commands are printed a couple of screens apart.
>
> This patch fixes all that, and also mentions "apropos" in the output
> of "help".
Thanks.
The idea sounds very good to me, but could you please post the output
of "help all" after applying the patch, so the improved results are
clearly visible? It's hard to judge that based only on the code
changes, and at least I cannot easily build a patched version where
I'm typing this.
> diff -u -r1.54 command.h
> --- command.h 17 Dec 2005 22:33:59 -0000 1.54
> +++ command.h 27 Oct 2006 19:39:27 -0000
> @@ -30,7 +30,7 @@
> enum command_class
> {
> /* Special args to help_list */
> - class_deprecated, all_classes = -2, all_commands = -1,
> + class_deprecated = -3, all_classes = -2, all_commands = -1,
> /* Classes of commands */
> no_class = -1, class_run = 0, class_vars, class_stack,
> class_files, class_support, class_info, class_breakpoint, class_trace,
I suggest to simply move class_deprecated after no_class, so that the
compiler gives it a value. We don't really care about the value in
that case, do we?