Re: [patch] Remove unimplemented MI commands [Re: Learn function name by its address]

Vladimir Prus
2009-06-28T06:45:10Z

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.

Thanks,
Volodya