RE: [patch v6 02/12] cli, btrace: add btrace cli

["Metzger, Markus T" <markus.t.metzger@intel.com>]

> -----Original Message-----
> From: Jan Kratochvil [mailto:jan.kratochvil@redhat.com]
> Sent: Tuesday, December 18, 2012 10:04 AM
> To: Metzger, Markus T
> Cc: palves@redhat.com; tromey@redhat.com; kettenis@gnu.org; gdb-patches@sourceware.org; markus.t.metzger@gmail.com
> Subject: Re: [patch v6 02/12] cli, btrace: add btrace cli
> 
> Hi Markus,
> 
> I find new class excessive, feature specific to a single arch has a new "help"
> main menu entry now.  For example existing - more powerful "record" feature is
> now under class_obscure.  Maybe it could be class_run but so far I would put
> it together to "record", that is a bit unfortunate class_obscure.

OK, I'll change the class to class_obscure.

Regards,
Markus.

> 
> 
> Thanks,
> Jan
> 
> 
> On Mon, 17 Dec 2012 17:01:31 +0100, markus.t.metzger@intel.com wrote:
> > diff --git a/gdb/command.h b/gdb/command.h
> > index 8eb86ba..b576b70 100644
> > --- a/gdb/command.h
> > +++ b/gdb/command.h
> > @@ -39,7 +39,7 @@ enum command_class
> >    no_class = -1, class_run = 0, class_vars, class_stack, class_files,
> >    class_support, class_info, class_breakpoint, class_trace,
> >    class_alias, class_bookmark, class_obscure, class_maintenance,
> > -  class_pseudo, class_tui, class_user, class_xdb,
> > +  class_pseudo, class_tui, class_user, class_xdb, class_btrace,
> >    no_set_class	/* Used for "show" commands that have no corresponding
> >  		   "set" command.  */
> >  };
> > diff --git a/gdb/testsuite/gdb.base/page.exp b/gdb/testsuite/gdb.base/page.exp
> > index 0629807..d9b3899 100644
> > --- a/gdb/testsuite/gdb.base/page.exp
> > +++ b/gdb/testsuite/gdb.base/page.exp
> > @@ -25,6 +25,7 @@ gdb_test_sequence "help" "unpaged help" {
> >      "List of classes of commands:"
> >      ""
> >      "aliases -- Aliases of other commands"
> > +    "branchtrace -- Recording a branch trace"
> >      "breakpoints -- Making program stop at certain points"
> >      "data -- Examining data"
> >      "files -- Specifying and examining files"
> > @@ -52,12 +53,12 @@ gdb_expect_list "paged help" \
> >      "List of classes of commands:"
> >      ""
> >      "aliases -- Aliases of other commands"
> > +    "branchtrace -- Recording a branch trace"
> >      "breakpoints -- Making program stop at certain points"
> >      "data -- Examining data"
> >      "files -- Specifying and examining files"
> >      "internals -- Maintenance commands"
> >      "obscure -- Obscure features"
> > -    "running -- Running the program"
> >  }
> >  gdb_test "q"
> >
> > --
> > 1.7.6.5
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895
Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052