> > + /* FIXME: There is a limitation in the deprecation mechanism, > > + and the warning ends up not being displayed for prefixed > > + aliases. So use a real command instead of an alias. > > + This is temporary code anyway, so should go away soon > > + after the next release cycle starts. */ > > I actually think that "temporary anyway" comment should be removed. We > don't really have a habit of removing deprecated commands. Marking a > command deprecated already makes it less visible. E.g., it removes the > command from completion suggestions. As long as the old command doesn't > get in the way, there's really no pressing need to remove it. > "soon after the next release" has itself a tendency to get > old and outdated :-) > > > This patch implements that change, as well as updates the GDB Manual. > > For now, it's a straight search-and-replace, as there was no real > > section where the option could go. Eventually, I think we will want > > to dissociate the options relative to the remote protocol, from the > > options specific to serial line handling. I think other option names > > might need to be renamed as well, but this I've already far exceeded > > the amount of time I could spend on this. OK, I removed the end of the comment entirely, even thought I've added an AI for the 7.8 release to remove the "alias". Naturally, I'm fine leaving it in for a while longer, if people prefer. > > gdb/ChangeLog: > > > > * cli/cli-cmds.c (show_baud_rate): Moved to serial.c as > > serial_baud_show_cmd. > > (_initialize_cli_cmds): Delete the code creating the > > "set/show remotebaud" commands. > > * serial.c (baud_rate): Move here from top.c. > > (serial_baud_show_cmd): Move here from cli/cli-cmds.c. > > (_initialize_serial): Create "set/show serial baud" commands. > > Add "set/show remotebaud" command aliases. > > * top.c (baud_rate): Moved to serial.c. > > * NEWS: Document the new "set/show serial baud" commands, > > replacing "set/show remotebaud". > > > > gdb/doc/ChangeLog: > > > > * gdb.texinfo: Replace "set remotebaud" and "show remotebaud" > > by "set serial baud" and "show serial baud" (resp) throughout. > > > > I hope it's a sufficient improvement in itself. OK to apply? > > This is fine with me. Great, thanks! Attached is what I ended up checking in. The only difference is the change in the comment. For Eli's comment about @w, I am about to send a followup-patch, with an explanation why. -- Joel