From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3460 invoked by alias); 4 May 2006 06:10:55 -0000 Received: (qmail 3452 invoked by uid 22791); 4 May 2006 06:10:55 -0000 X-Spam-Check-By: sourceware.org Received: from zigzag.lvk.cs.msu.su (HELO zigzag.lvk.cs.msu.su) (158.250.17.23) by sourceware.org (qpsmtpd/0.31) with ESMTP; Thu, 04 May 2006 06:10:54 +0000 Received: from Debian-exim by zigzag.lvk.cs.msu.su with spam-scanned (Exim 4.50) id 1FbX35-0004LE-1v for gdb-patches@sources.redhat.com; Thu, 04 May 2006 10:10:52 +0400 Received: from zigzag.lvk.cs.msu.su ([158.250.17.23]) by zigzag.lvk.cs.msu.su with esmtp (Exim 4.50) id 1FbX2z-0004K4-OO; Thu, 04 May 2006 10:10:45 +0400 From: Vladimir Prus To: "Jim Blandy" Subject: Re: Variable objects: references formatting Date: Thu, 04 May 2006 06:10:00 -0000 User-Agent: KMail/1.7.2 Cc: gdb-patches@sources.redhat.com References: <200605040918.33856.ghost@cs.msu.su> <8f2776cb0605032308r7645469di45fda7e57ae5718c@mail.gmail.com> In-Reply-To: <8f2776cb0605032308r7645469di45fda7e57ae5718c@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200605041010.44975.ghost@cs.msu.su> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-05/txt/msg00032.txt.bz2 On Thursday 04 May 2006 10:08, Jim Blandy wrote: > On 5/3/06, Vladimir Prus wrote: > > > - In GDB, when traversing types, remember to call check_typedef to > > > avoid having your traversal stopped by typedef nodes. > > > > Thanks, noted. So, in this case I should have called check_typedef before > > checking if type is reference, right? > > Well, the get_type function is taking care of that one for you. So > it's enough to check the new types you uncover with TYPE_TARGET_TYPE. Ok, I see. > > > Would you like me to resend the patch adjusted per your comments, or > > you've already done those changes locally? > > I've already done them. Here's the patch I actually committed (which > I should have posted when I did so): Thanks you! - Volodya