From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21337 invoked by alias); 28 Mar 2007 19:17:44 -0000 Received: (qmail 21297 invoked by uid 22791); 28 Mar 2007 19:17:42 -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; Wed, 28 Mar 2007 20:17:35 +0100 Received: from localhost (localhost [127.0.0.1]) by filtered-nile.gnat.com (Postfix) with ESMTP id E304B48CC42; Wed, 28 Mar 2007 15:17:33 -0400 (EDT) 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 18857-01-6; Wed, 28 Mar 2007 15:17:33 -0400 (EDT) Received: from joel.gnat.com (unknown [70.71.0.212]) by nile.gnat.com (Postfix) with ESMTP id 56A3448CBED; Wed, 28 Mar 2007 15:17:33 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 18BFEE7B43; Wed, 28 Mar 2007 15:18:45 -0400 (EDT) Date: Wed, 28 Mar 2007 19:17:00 -0000 From: Joel Brobecker To: gcc-patches@gcc.gnu.org Cc: gdb-patches@sourceware.org Subject: [RFA/libiberty] Enhance FILENAME_CMP for Windows filesystems Message-ID: <20070328191845.GB3956@adacore.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="DocE+STaALJfprDB" Content-Disposition: inline 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-03/txt/msg00291.txt.bz2 --DocE+STaALJfprDB Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-length: 2236 Hello, We are using the FILENAME_CMP macro in GDB quite a bit, and it is currently defined in include/filenames.h as: #define FILENAME_CMP(s1, s2) strcasecmp(s1, s2) This is for "DOSish" filesystems. For other filesystems, it's defined as a straight strcmp: #define FILENAME_CMP(s1, s2) strcmp(s1, s2) I came across a case where the DWARF debug info unfortunately used '\' as the directory separator in one case (in the .debug_info section), while using '/' in another case (in the .debug_line section). As a result, FILENAME_CMP returned that the following two files were not equal: c:\cygwin\home\brobecke\tgdb\ex\\foo.adb c:/cygwin/home/brobecke/tgdb/ex//foo.adb This causes the following problem in GDB, when we compile the unit by giving GCC the full path name to the associated file, such as: % gcc -c -g c:\full\path\to\foo.adb The GDB error looks like this: % gdb foo (gdb) b foo.adb:3 No line 3 in file "c:\cygwin\home\brobecke\tgdb\ex\\foo.adb". I propose to add a new filename_cmp function to libiberty: - On Unix, no change, it calls strcmp - On Windows, we make slash and backslash equal. This is a modest improvement, since there are many other things we can do to enhance it (such as normalizing the path so that '//' and '/' are treated as equal for instance). But this paves the path for further improvements of that sort. And in the meantime, if fixes the problem we are facing. For that, I added a new file, filename_cmp.c, updated the Makefile to add this file to the list of required objects, and then changed the FILENAME_CMP define to unconditionally call this new function. libiberty/ change: 2007-03-28 Joel Brobecker * filename_cmp.c: New file. * Makefile.in (CFILES): Add filename_cmp.c. (REQUIRED_OFILES): Add filename_cmp.o (filename_cmp.o): New rule. include/ change: 2007-03-28 Joel Brobecker * filenames.h (FILENAME_CMP): Adjust define to call filename_cmp regardless of the type of file system. Tested on x86-linux and x86-windows, without any regression (using the GDB testsuite). OK to apply? Thank you, -- Joel --DocE+STaALJfprDB Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="filename_cmp.c" Content-length: 1959 /* File name comparison routine. * Copyright (C) 2007 Free Software Foundation, Inc. This file is part of the libiberty library, which is part of GCC. This file is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version. In addition to the permissions in the GNU General Public License, the Free Software Foundation gives you unlimited permission to link the compiled version of this file into combinations with other programs, and to distribute those combinations without any restriction coming from the use of this file. (The General Public License restrictions do apply in other respects; for example, they cover modification of the file, and distribution when not linked into a combined executable.) This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with this program; if not, write to the Free Software Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA. */ #ifdef HAVE_STRING_H #include #endif #include #include "filenames.h" int filename_cmp (const char *s1, const char *s2) { #ifndef HAVE_DOS_BASED_FILE_SYSTEM return strcmp(s1, s2); #else for (;;) { int c1 = tolower (*s1); int c2 = tolower (*s2); /* On DOS-based file systems, the '/' and the '\' are equivalent. */ if (c1 == '/') c1 = '\\'; if (c2 == '/') c2 = '\\'; if (c1 != c2) return (c1 - c2); if (c1 == '\0') return 0; s1++; s2++; } #endif } --DocE+STaALJfprDB Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="filename_cmp.diff" Content-length: 3385 Index: libiberty/Makefile.in =================================================================== RCS file: /cvs/src/src/libiberty/Makefile.in,v retrieving revision 1.85 diff -u -p -r1.85 Makefile.in --- libiberty/Makefile.in 5 Mar 2007 22:15:38 -0000 1.85 +++ libiberty/Makefile.in 28 Mar 2007 19:16:41 -0000 @@ -1,6 +1,6 @@ # # Makefile -# Copyright (C) 1990, 91-99, 2000, 2001, 2002, 2003, 2004, 2005, 2006 +# Copyright (C) 1990, 91-99, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007 # Free Software Foundation # # This file is part of the libiberty library. @@ -131,8 +131,8 @@ CFILES = alloca.c argv.c asprintf.c atex calloc.c choose-temp.c clock.c concat.c cp-demangle.c \ cp-demint.c cplus-dem.c \ dyn-string.c \ - fdmatch.c ffs.c fibheap.c floatformat.c fnmatch.c \ - fopen_unlocked.c \ + fdmatch.c ffs.c fibheap.c filename_cmp.c floatformat.c \ + fnmatch.c fopen_unlocked.c \ getcwd.c getopt.c getopt1.c getpagesize.c getpwd.c getruntime.c \ gettimeofday.c \ hashtab.c hex.c \ @@ -165,8 +165,8 @@ REQUIRED_OFILES = ./regex.o ./cplus-dem. ./alloca.o ./argv.o \ ./choose-temp.o ./concat.o ./cp-demint.o \ ./dyn-string.o \ - ./fdmatch.o ./fibheap.o ./floatformat.o ./fnmatch.o \ - ./fopen_unlocked.o \ + ./fdmatch.o ./fibheap.o ./filename_cmp.o ./floatformat.o \ + ./fnmatch.o ./fopen_unlocked.o \ ./getopt.o ./getopt1.o ./getpwd.o ./getruntime.o \ ./hashtab.o ./hex.o \ ./lbasename.o ./lrealpath.o \ @@ -651,6 +651,12 @@ $(CONFIGURED_OFILES): stamp-picdir else true; fi $(COMPILE.c) $(srcdir)/fibheap.c $(OUTPUT_OPTION) +./filename_cmp.o: $(srcdir)/filename_cmp.c $(INCDIR)/filenames.h + if [ x"$(PICFLAG)" != x ]; then \ + $(COMPILE.c) $(PICFLAG) $(srcdir)/filename_cmp.c -o pic/$@; \ + else true; fi + $(COMPILE.c) $(srcdir)/filename_cmp.c $(OUTPUT_OPTION) + ./floatformat.o: $(srcdir)/floatformat.c config.h $(INCDIR)/ansidecl.h \ $(INCDIR)/floatformat.h $(INCDIR)/libiberty.h if [ x"$(PICFLAG)" != x ]; then \ Index: include/filenames.h =================================================================== RCS file: /cvs/src/src/include/filenames.h,v retrieving revision 1.3 diff -u -p -r1.3 filenames.h --- include/filenames.h 10 May 2005 10:21:08 -0000 1.3 +++ include/filenames.h 28 Mar 2007 19:16:41 -0000 @@ -5,7 +5,7 @@ use forward- and back-slash in path names interchangeably, and some of them have case-insensitive file names. - Copyright 2000, 2001 Free Software Foundation, Inc. + Copyright 2000, 2001, 2007 Free Software Foundation, Inc. This file is part of BFD, the Binary File Descriptor library. @@ -38,14 +38,15 @@ Foundation, Inc., 51 Franklin Street - F want to know whether to prepend the current working directory to a file name, which should not be done with a name like d:foo. */ #define IS_ABSOLUTE_PATH(f) (IS_DIR_SEPARATOR((f)[0]) || (((f)[0]) && ((f)[1] == ':'))) -#define FILENAME_CMP(s1, s2) strcasecmp(s1, s2) #else /* not DOSish */ #define IS_DIR_SEPARATOR(c) ((c) == '/') #define IS_ABSOLUTE_PATH(f) (IS_DIR_SEPARATOR((f)[0])) -#define FILENAME_CMP(s1, s2) strcmp(s1, s2) #endif /* not DOSish */ +extern int filename_cmp (const char *s1, const char *s2); +#define FILENAME_CMP(s1, s2) filename_cmp(s1, s2) + #endif /* FILENAMES_H */ --DocE+STaALJfprDB--