From: Jim Ingham <jingham@apple.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: Aleksandar Ristovski <ARistovski@qnx.com>,
Vladimir Prus <ghost@cs.msu.su>,
gdb@sourceware.org
Subject: Re: MI varobj artificial fields
Date: Wed, 16 Apr 2008 19:34:00 -0000 [thread overview]
Message-ID: <4E580E6F-1A74-489C-8825-C28F37BA9B55@apple.com> (raw)
In-Reply-To: <20080416183646.GA21807@caradoc.them.org>
I assumed that in cases where the protections were interleaved it was
just cruft of history, and if you were going to see protections at
all, it would make more sense to put them in just three groups. If
you have turn-outs, then of course it makes more sense to have three,
since otherwise you do a little "did I turn out the right private"
dance which is pretty annoying. There probably isn't one correct
answer to this question.
Jim
On Apr 16, 2008, at 11:36 AM, Daniel Jacobowitz wrote:
> On Wed, Apr 16, 2008 at 11:21:53AM -0700, Jim Ingham wrote:
>> Yeah, I think this was just added so you get the organization for
>> free.
>> Note that if you go switch to an attribute, the UI is going to have
>> to
>> reorder the variables to get all the private ones together, etc.
>
> Is that really what you'd want? GDB's ptype will group things by
> protection in the order they're present anyway, repeating protections
> if that's what the source did. I think this is much more logical.
>
> class foo
> {
> public:
> int a;
>
> private:
> int b;
>
> public:
> int c;
> };
>
> --
> Daniel Jacobowitz
> CodeSourcery
next prev parent reply other threads:[~2008-04-16 18:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4806400B.7050905@qnx.com>
2008-04-16 19:16 ` Jim Ingham
2008-04-16 19:22 ` Daniel Jacobowitz
2008-04-16 19:34 ` Jim Ingham [this message]
2008-04-16 22:05 ` Pedro Alves
2008-04-18 10:22 ` André Pönitz
2008-04-16 16:27 Vladimir Prus
2008-04-16 18:06 ` Pedro Alves
2008-04-16 18:18 ` André Pönitz
2008-04-16 18:22 ` Marc Khouzam
2008-04-16 18:51 ` Jim Ingham
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=4E580E6F-1A74-489C-8825-C28F37BA9B55@apple.com \
--to=jingham@apple.com \
--cc=ARistovski@qnx.com \
--cc=drow@false.org \
--cc=gdb@sourceware.org \
--cc=ghost@cs.msu.su \
/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