From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14377 invoked by alias); 14 Sep 2009 11:24:13 -0000 Received: (qmail 14367 invoked by uid 22791); 14 Sep 2009 11:24:12 -0000 X-SWARE-Spam-Status: No, hits=-2.8 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from lo.gmane.org (HELO lo.gmane.org) (80.91.229.12) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 14 Sep 2009 11:24:08 +0000 Received: from list by lo.gmane.org with local (Exim 4.50) id 1Mn9eI-0008EL-Q8 for gdb-patches@sources.redhat.com; Mon, 14 Sep 2009 13:23:10 +0200 Received: from h86-62-88-129.ln.rinet.ru ([86.62.88.129]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 14 Sep 2009 13:23:10 +0200 Received: from vladimir by h86-62-88-129.ln.rinet.ru with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Mon, 14 Sep 2009 13:23:10 +0200 To: gdb-patches@sources.redhat.com From: Vladimir Prus Subject: Re: Patch: implement new dynamic varobj spec Date: Mon, 14 Sep 2009 11:24:00 -0000 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit User-Agent: KNode/0.10.9 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: 2009-09/txt/msg00394.txt.bz2 Tom Tromey wrote: > This is the long-awaited dynamic varobj patch. > It implements the spec as described by Vladimir: > > http://sourceware.org/ml/gdb/2009-07/msg00088.html > > .. with various minor changes we've discussed on that thread and > elsewhere. I didn't give the follow-up URLs; I think that the doc patch > ought to be sufficient, and if it isn't, I will fix it up. > > This has been in development for quite a while, including testing using > two different MI consumers (Vladimir's and one by Noam Yorav-Raphael). > > There are three outstanding bugs. > > * A dynamic varobj bug that I couldn't reproduce > http://sourceware.org/ml/archer/2009-q3/msg00169.html > > * Lazy string printing. > http://sourceware.org/ml/archer/2009-q3/msg00189.html > I'm not sure what we'll do about this in the 7.0 timeframe, maybe > nothing. > > * -var-info-path-expression doesn't work > http://sourceware.org/bugzilla/show_bug.cgi?id=10252 > (I am not sure it can be made to work, I was planning to turn it into > an error.) > > > This patch needs a doc review. > > This patch doesn't apply directly due to the big python renaming. > You can probably run sed on the patch if you care to try it. For avoidance of doubt, I think this patch should go in. I plan to read the doc changes, and glance at the code, but I am fairly confident that this is overall good. - Volodya