From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2421 invoked by alias); 3 Dec 2008 00:04:19 -0000 Received: (qmail 1950 invoked by uid 22791); 3 Dec 2008 00:04:14 -0000 X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 03 Dec 2008 00:03:13 +0000 Received: (qmail 6381 invoked from network); 3 Dec 2008 00:03:11 -0000 Received: from unknown (HELO orlando.local) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 3 Dec 2008 00:03:11 -0000 From: Pedro Alves To: gdb-patches@sourceware.org Subject: Re: Patch to support spaces in filenames & paths Date: Wed, 03 Dec 2008 00:04:00 -0000 User-Agent: KMail/1.9.10 Cc: Daniel Jacobowitz , Michael Snyder , Jon Beniston References: <7D653316E38B4305941199D722BF20B0@bibi> <4935A5E2.5050503@vmware.com> <20081202233738.GA15530@caradoc.them.org> In-Reply-To: <20081202233738.GA15530@caradoc.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200812030003.48011.pedro@codesourcery.com> 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: 2008-12/txt/msg00042.txt.bz2 On Tuesday 02 December 2008 23:37:38, Daniel Jacobowitz wrote: > On Tue, Dec 02, 2008 at 01:17:22PM -0800, Michael Snyder wrote: > > Denis, is it possible that Jon's patch will serve in place of > > the filename portion of your patch? And that you could then > > resubmit your patch with just the other portions? > > > > Jon's patch: > > http://sourceware.org/ml/gdb-patches/2008-12/msg00032.html > > > > Denis' patch: > > http://sourceware.org/ml/gdb-patches/2008-12/msg00029.html > > I have not looked at the patches in depth but I encourage Denis's > approach - uniform parsing is a Very Good Thing and buildargv is what > we use elsewhere. Same here, but I think it should be using the new gdb_buildargv. -- Pedro Alves