Mirror of the gdb mailing list
 help / color / mirror / Atom feed
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


  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