From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 14071 invoked by alias); 8 May 2005 13:43:45 -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 14025 invoked by uid 22791); 8 May 2005 13:43:38 -0000 Received: from lakermmtao11.cox.net (HELO lakermmtao11.cox.net) (68.230.240.28) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Sun, 08 May 2005 13:43:38 +0000 Received: from white ([68.9.64.121]) by lakermmtao11.cox.net (InterMail vM.6.01.04.00 201-2131-118-20041027) with ESMTP id <20050508134226.BUGJ28600.lakermmtao11.cox.net@white> for ; Sun, 8 May 2005 09:42:26 -0400 Received: from bob by white with local (Exim 3.35 #1 (Debian)) id 1DUm37-0001aW-00 for ; Sun, 08 May 2005 09:42:25 -0400 Date: Sun, 08 May 2005 13:43:00 -0000 From: Bob Rossi To: gdb-patches@sources.redhat.com Subject: Re: fullname syntax variable for testsuite Message-ID: <20050508134225.GA6099@white> Mail-Followup-To: gdb-patches@sources.redhat.com References: <20050505173202.GC31758@white> <20050505173942.GA17561@nevyn.them.org> <20050506004415.GD481@white> <20050506015442.GA3874@nevyn.them.org> <20050507031258.GA4274@white> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050507031258.GA4274@white> User-Agent: Mutt/1.3.28i X-SW-Source: 2005-05/txt/msg00187.txt.bz2 On Fri, May 06, 2005 at 11:12:58PM -0400, Bob Rossi wrote: > On Thu, May 05, 2005 at 09:54:42PM -0400, Daniel Jacobowitz wrote: > > On Thu, May 05, 2005 at 08:44:15PM -0400, Bob Rossi wrote: > > > Here is the updated patch. I broke down the regexp into several cases. > > > It should be very easy to maintain (add/remove) the regexp as time goes > > > on. If there are any other modifications that need to be made, please > > > let me know. > > > > > > Thanks, > > > Bob Rossi > > > > > > > > > 2005-05-05 Bob Rossi > > > > > > * lib/gdb.exp (fullname_syntax): Added global regex. > > > (fullname_syntax_POSIX): Ditto. > > > (fullname_syntax_UNC): Ditto. > > > (fullname_syntax_DOS_CASE): Ditto. > > > (fullname_syntax_DOS): Ditto. > > > * gdb.mi/mi-file.exp (test_file_list_exec_source_file): Added > > > fullname_syntax variable to scope and use it. > > > (test_file_list_exec_source_files): Ditto. > > > * gdb.mi/mi2-file.exp (test_file_list_exec_source_file): Ditto. > > > > This patch is OK with me. Please wait a day to see if anyone else has > > comments, and then you can check this in. After you do that I'll > > revise Dennis's patch. > > I changed the DOS_CASE > from set fullname_syntax_DOS_CASE {\\[^\\]+\\} > to set fullname_syntax_DOS_CASE {\\[^\\].*\\} > as Dennis noticed, it didn't match the regex > \a\b\c\d\basics.c > > Anyways, please consider this as a last chance to speak up with any > problems. Otherwise, I'll commit tomorrow. Committed. Bob Rossi