This is the mail archive of the gdb-patches@sourceware.org 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: pr 11067 patch


On 02/19/10 09:36, Jan Kratochvil wrote:
Hi Chris,

On Fri, 19 Feb 2010 15:28:46 +0100, Joel Brobecker wrote:
Jan's question, IMO (Jan, please correct me if I am wrong), were meant as
a discussion starter, rather than an objection.

yes. As I said in practice it is an improvement even for MI (at least for Eclipse CDT (CDI)) but formally I do not find it right for MI (as such diplay feature should be done by MI frontend and not the MI backend IMO).

IMHO I would do the change for CLI but not for the MI.

The problem is that I don't know any way to change the enum formatting for CLI but leave it alone for MI-. That's one of the reasons I proposed the "set enum-fmt" solution to, avoid breaking historical default behaviour but still giving users who wanted it the option to use a more informative format.


I'll be glad to dust the patch off--I haven't deleted either version--if people want it, and I'll even look around for some way to distinguish between running under CLI vs. MI if that's the right thing to do. It had just gotten to the point where making a really trivial change was taking way more effort than it was worth.

Chris

  But besides no
approval rights I am even not a regular MI/frontend user.  So just my $0.02.


Regards,
Jan


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