This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Remove unimplemented MI commands [Re: Learn function name by its address]
- From: Stan Shebs <stan at codesourcery dot com>
- To: Vladimir Prus <vladimir at codesourcery dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Wed, 01 Jul 2009 12:13:55 -0700
- Subject: Re: [patch] Remove unimplemented MI commands [Re: Learn function name by its address]
- References: <8ac60eac0906271144k61bbb6e3sc092d2780dc4192e@mail.gmail.com> <h273ev$glu$2@ger.gmane.org>
Vladimir Prus wrote:
Paul Pluzhnikov wrote:
On Fri, Jun 26, 2009 at 3:41 PM, Joel Brobecker<brobecker@adacore.com> wrote:
I tend to think that if nobody has implemented these in $X years, then
nobody ever will. But then, I'm generally also in the "never comment
out code, just delete it" camp.
So am I. The code is still around, just in CVS instead of inside
a comment. Leaving it commented out just creates more clutter that
tends to stay rather than being fixed. So I really favor deleting.
Ok, patch attached.
The code patch is fine. Although at least one of those commands is likely
to be implemented within a year (-break-commands), there's no point to
keep it recognized-but-not-implemented for now.
Also CodeSourcery has a contract to implement MI versions of tracepoint
commands, but we might as well start fresh, rather than try to preserve
a few lines of ancient code.
Stan