From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30555 invoked by alias); 23 Jan 2007 23:59:09 -0000 Received: (qmail 30547 invoked by uid 22791); 23 Jan 2007 23:59:08 -0000 X-Spam-Check-By: sourceware.org Received: from nile.gnat.com (HELO nile.gnat.com) (205.232.38.5) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 23 Jan 2007 23:59:02 +0000 Received: from localhost (localhost [127.0.0.1]) by filtered-nile.gnat.com (Postfix) with ESMTP id 59E6F48CD9E for ; Tue, 23 Jan 2007 18:59:00 -0500 (EST) Received: from nile.gnat.com ([127.0.0.1]) by localhost (nile.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 02401-01-4 for ; Tue, 23 Jan 2007 18:59:00 -0500 (EST) Received: from takamaka.act-europe.fr (joel.gnat.com [205.232.38.116]) by nile.gnat.com (Postfix) with ESMTP id 36EB348CBE2 for ; Tue, 23 Jan 2007 18:59:00 -0500 (EST) Received: by takamaka.act-europe.fr (Postfix, from userid 1000) id 5150734C099; Wed, 24 Jan 2007 03:59:58 +0400 (RET) Date: Tue, 23 Jan 2007 23:59:00 -0000 From: Joel Brobecker To: gdb-patches@sourceware.org Subject: Re: [RFA/dwarf2read.c] Make Ada structs "typedef"'ed implicitly Message-ID: <20070123235958.GA4676@adacore.com> References: <20070111160458.GM23012@adacore.com> <20070121172942.GF12463@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20070121172942.GF12463@nevyn.them.org> User-Agent: Mutt/1.4.2.2i 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: 2007-01/txt/msg00486.txt.bz2 > > 2007-01-11 Joel Brobecker > > > > * dwarf2read.c (add_partial_symbol): Create an extra partial > > symbol in the VAR_DOMAIN for Ada structures, unions or enums. > > (new_symbol): Likewise for symbols. > > > > Tested on x86-linux. 1 extra PASS in gdb.ada/null_record.exp, no > > regression. > > > > OK to apply? > > OK, thanks! On to the next :-) Thanks! Now checked in. > > - || cu->language == language_java) > > + || cu->language == language_java > > + || cu->language == language_ada) > > Tab/space mixup? Oops, you're right. Fixed before checking the patch in. Thanks Daniel, -- Joel