Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}"

Tom Tromey
2012-05-17T15:27:47Z

>>>>> "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