This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: add_setshow_cmd_full


>>>>> "Andrew" == Andrew Cagney <ac131313@cygnus.com> writes:

Andrew> Please be careful to configure GDB with
Andrew> --enable-gdb-build-warnings=,-Werror so that it is compiled
Andrew> with the -Werror flag.

Thanks for catching this.  I'm rebuilding with that right now; I wired
it into my standard build script so I won't make this mistake again.

Andrew> Don't forget to remove they stray declaration:

>> +extern struct cmd_list_element *add_setshow_cmd (char *name,
>> +						 enum command_class class,
>> +						 var_types var_type, void *var,
>> +						 char *set_doc, char *show_doc,

Andrew> from the change.

That declaration is part of the change, since I wanted to make
add_setshow_cmd public, and add add_setshow_cmd_full as well.  The
latter is only used in a few places.

Would you prefer I use add_setshow_cmd_full everywhere, and just have
90% of the calls have `NULL, NULL' at the end?  I can do that...

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]