From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14297 invoked by alias); 3 Oct 2004 16:28:44 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 14290 invoked from network); 3 Oct 2004 16:28:43 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 3 Oct 2004 16:28:43 -0000 Received: from drow by nevyn.them.org with local (Exim 4.34 #1 (Debian)) id 1CE9E3-0001CR-8Q; Sun, 03 Oct 2004 12:28:43 -0400 Date: Sun, 03 Oct 2004 16:28:00 -0000 From: Daniel Jacobowitz To: Jim Blandy Cc: gdb-patches@sources.redhat.com Subject: Re: [rfa/dwarf] Support for attributes pointing to a different CU Message-ID: <20041003162843.GC3234@nevyn.them.org> Mail-Followup-To: Jim Blandy , gdb-patches@sources.redhat.com References: <20040923045723.GA11871@nevyn.them.org> <20040924003412.GB10500@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.5.1+cvs20040105i X-SW-Source: 2004-10/txt/msg00038.txt.bz2 On Tue, Sep 28, 2004 at 06:05:18PM -0500, Jim Blandy wrote: > > If you made your queue into a real queue, with head and tail pointers, > and had queue_comp_unit not re-queue things that were already queued, > then you could replace the two nested loops around load_full_comp_unit > with a single, straightforward linked list walk, couldn't you? Yes, it looks like that will work. Much nicer. -- Daniel Jacobowitz