From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15146 invoked by alias); 20 Dec 2010 06:38:40 -0000 Received: (qmail 15064 invoked by uid 22791); 20 Dec 2010 06:38:39 -0000 X-SWARE-Spam-Status: No, hits=-0.9 required=5.0 tests=AWL,BAYES_40 X-Spam-Check-By: sourceware.org Received: from ms9.webland.ch (HELO ms9smtp.webland.ch) (92.43.217.109) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 20 Dec 2010 06:38:12 +0000 Received: from macserver.private ([84.74.43.109]) by ms9smtp.webland.ch (Webland Mail Server v10.0) with ASMTP id CSG76008; Mon, 20 Dec 2010 07:38:08 +0100 Received: from localhost (localhost [127.0.0.1]) by macserver.private (Postfix) with ESMTP id D15ABA6E863; Mon, 20 Dec 2010 07:38:06 +0100 (CET) Received: from macserver.private ([127.0.0.1]) by localhost (macserver.private [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id QazAtsUFqUwq; Mon, 20 Dec 2010 07:38:06 +0100 (CET) Received: from [192.168.1.86] (unknown [192.168.1.86]) by macserver.private (Postfix) with ESMTP id 5195CA6E858; Mon, 20 Dec 2010 07:38:06 +0100 (CET) Message-ID: <4D0EF9CE.80402@indel.ch> Date: Mon, 20 Dec 2010 06:38:00 -0000 From: Raphael Zulliger User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.13) Gecko/20101207 Lightning/1.0b2 Thunderbird/3.1.7 MIME-Version: 1.0 To: Eli Zaretskii CC: gdb-patches@sourceware.org Subject: Re: proposal: substitute-path handles foreign dir separators References: <4D0B7125.2010203@indel.ch> <834oach14b.fsf@gnu.org> In-Reply-To: <834oach14b.fsf@gnu.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes 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: 2010-12/txt/msg00362.txt.bz2 On 17.12.2010 16:26, Eli Zaretskii wrote: > There were discussions on this a few months ago, you may wish to look > them up. I just tried to look it up but I couldn't find the particular discussion unless you refer to the one mentioned by Pedor Alves: > This isn't the right way, for 2 reasons: > > . It makes the code uglier than it must be. > > . The support for DOS-style file names is based on compile-time > macros, so it cannot be turned on and off. Thus, you leave no fire > escape for Unix users who just happen to have file names with > literal backslashes, improbable as that may be. I agree with both. I didn't thought about the case of using escaped file names on Unix systems. > In this part, I simply don't understand why you needed the > HAVE_DOS_BASED_FILE_SYSTEM branch. DOS/Windows file-system calls > support forward slashes just fine, so there's no need to rewrite the > slashes. Thanks. Somehow I wasn't aware of that. > Finally, please wait for other opinions, as I'm not sure mine is in > consensus. > I think we should forget about my patch as I posted it. Let's continue the discussion based on Pedros patch instead. Thanks. Raphael