Re: [RFA, doc RFA] set print frame-arguments-raw on|off

[Tom Tromey <tromey@redhat.com>]

>>>>> "Doug" == Doug Evans <dje@google.com> writes:

Doug> Here is a revised version of "set print frame-arguments-raw".
Doug> It renames the option to "set print raw frame-arguments" to allow for
Doug> other things that we might want to control raw printing of.

Looks good to me.

It might be nice to allow this as an argument to "bt" as well, for
one-off uses.

Doug> There's one thing I'm not sure of:
Doug> which one of gdbcmd.h vs cli/cli-cmds.h to use?
Doug> They each declare the set/show list globals.
Doug> gdbcmd.h has a comment saying it's deprecated.
Doug> However, it defines maintenance_{set,show}_cmdlist whereas cli-cmds.h
Doug> does not (which I agree with to the extent that if I were to try to clean
Doug> this area up I would make a few changes, e.g., better separation of gdb
Doug> and cli-as-module, but they're beyond the scope of this patch).

I think cli/cli-cmds.h is preferable for "pure CLI" things.
IMNSHO any public object ought to have a single declaration in a single
header file.  Any kind of duplication here is bad.

Tom