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: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"

From: Tom Tromey <tromey at redhat dot com>
To: AndrÃ PÃnitz <andre dot poenitz at mathematik dot tu-chemnitz dot de>
Cc: xgsa <xgsa at yandex dot ua>, "gdb-patches\ at sourceware dot org" <gdb-patches at sourceware dot org>, Vladimir_Prus at mentor dot com
Date: Thu, 17 May 2012 09:27:17 -0600
Subject: Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"
References: <305151335125397@web28e.yandex.ru> <87likttnme.fsf@fleche.redhat.com> <20120516010500.GB3321@klara.mpi.htwm.de> <87zk98qijh.fsf@fleche.redhat.com> <20120517121923.GB6350@klara.mpi.htwm.de>

>>>>> "AndrÃ" == AndrÃ PÃnitz <andre.poenitz@mathematik.tu-chemnitz.de> writes:

AndrÃ> 'git blame' thinks you added it in September 2009, so gdb 7.0 is
AndrÃ> probably the first one that has it.  Calling code "broken" just
AndrÃ> because it handles (also...) "legacy" setups is a bit of a stretch
AndrÃ> in my opinion.

We're only talking about dynamic varobjs here, which the MI user must
explicitly request.  Checking a dynamic varobj for "{...}" has always
been wrong, since they were first introduced.

Looking at this for non-dynamic varobjs... well, I wouldn't do that, but
it isn't relevant.

AndrÃ> If you really want to read more into the comment than what was
AndrÃ> actually written, then perhaps a friendly nudge to adapt the
AndrÃ> documentation to existing practice.

As far as I know the documentation is correct.

Tom

References:
- Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"
  - From: Tom Tromey
- Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"
  - From: André Pönitz
- Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"
  - From: Tom Tromey
- Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"
  - From: André Pönitz

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