From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6004 invoked by alias); 17 May 2012 12:19:41 -0000 Received: (qmail 5983 invoked by uid 22791); 17 May 2012 12:19:40 -0000 X-SWARE-Spam-Status: No, hits=-3.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,KHOP_THREADED,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from cora.hrz.tu-chemnitz.de (HELO cora.hrz.tu-chemnitz.de) (134.109.228.40) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 17 May 2012 12:19:26 +0000 Received: from p5ddc8719.dip0.t-ipconnect.de ([93.220.135.25] helo=localhost) by cora.hrz.tu-chemnitz.de with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.76) (envelope-from ) id 1SUzfw-0000aq-Us; Thu, 17 May 2012 14:19:25 +0200 Date: Thu, 17 May 2012 12:19:00 -0000 From: =?iso-8859-1?Q?Andr=E9_P=F6nitz?= To: Tom Tromey Cc: xgsa , "gdb-patches@sourceware.org" , Vladimir_Prus@mentor.com Subject: Re: [patch] Use the string returned by pretty printer for MI varobjs instead of "{...}" Message-ID: <20120517121923.GB6350@klara.mpi.htwm.de> References: <305151335125397@web28e.yandex.ru> <87likttnme.fsf@fleche.redhat.com> <20120516010500.GB3321@klara.mpi.htwm.de> <87zk98qijh.fsf@fleche.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <87zk98qijh.fsf@fleche.redhat.com> X-Scan-AV: cora.hrz.tu-chemnitz.de;2012-05-17 14:19:25;ce8b05d456bb9f2f6d525034bb92d689 X-Scan-SA: cora.hrz.tu-chemnitz.de;2012-05-17 14:19:25;e5ee0b782c7827207bae6606b33c9a2c X-Spam-Score: -1.0 (-) X-Spam-Report: --- Textanalyse SpamAssassin 3.3.1 (-1.0 Punkte) Fragen an/questions to: Postmaster TU Chemnitz * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP --- Ende Textanalyse X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-05/txt/msg00648.txt.bz2 On Wed, May 16, 2012 at 08:58:42AM -0600, Tom Tromey wrote: > >>>>> writes: > > Tom> My recollection is that the code originally worked this way, but > Tom> Vladimir asked for the "{...}" behavior specifically. > > André> It might affect frontends taking a "{...}" as indication that > André> there are childrens. In fact, I just found some code using that > André> as a shortcut. > > That code is just plain broken, though. The has_more field conveys > this information. If you say so. 'git blame' thinks you added it in September 2009, so gdb 7.0 is probably the first one that has it. Calling code "broken" just because it handles (also...) "legacy" setups is a bit of a stretch in my opinion. In any case, I was merely trying to convey the idea that an unsuspecting frontend implementor might read the documentation in a way that would be incompatible with the change. I am personally not affected by any change in the -var-* output, so maybe I should not have bothered to comment at all. If you really want to read more into the comment than what was actually written, then perhaps a friendly nudge to adapt the documentation to existing practice. Andre'