From: Chris Moller <cmoller@redhat.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Cc: Joel Brobecker <brobecker@adacore.com>, gdb-patches@sourceware.org
Subject: Re: pr 11067 patch
Date: Fri, 19 Feb 2010 14:54:00 -0000 [thread overview]
Message-ID: <4B7EA5F9.6030001@redhat.com> (raw)
In-Reply-To: <20100219143609.GA1210@host0.dyn.jankratochvil.net>
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
>
next prev parent reply other threads:[~2010-02-19 14:54 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-11 2:55 Chris Moller
2010-02-11 9:30 ` Joel Brobecker
2010-02-11 14:19 ` Chris Moller
2010-02-11 19:50 ` Tom Tromey
2010-02-12 4:11 ` Joel Brobecker
2010-02-12 15:48 ` Chris Moller
2010-02-13 11:49 ` Jan Kratochvil
2010-02-13 18:56 ` Chris Moller
2010-02-19 14:28 ` Joel Brobecker
2010-02-19 14:36 ` Jan Kratochvil
2010-02-19 14:45 ` Joel Brobecker
2010-02-19 14:54 ` Chris Moller [this message]
2010-02-19 18:50 ` Jan Kratochvil
2010-02-19 19:52 ` Chris Moller
2010-02-19 20:11 ` Jan Kratochvil
2010-02-22 9:22 ` Vladimir Prus
2010-02-23 23:55 ` Tom Tromey
2010-03-11 15:44 ` pr 11067 patch resurrected from the dead Chris Moller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B7EA5F9.6030001@redhat.com \
--to=cmoller@redhat.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
--cc=jan.kratochvil@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox