From: Johannes Sixt Date: Thu, 08 Oct 2009 08:10:53 GMT Subject: Re: [PATCH 1/3] completion: fix alias listings with newlines Message-ID: <4ACD9E8D.1070702@viscovery.net> In-Reply-To: <4ACD9C66.1080308@gmail.com> Stephen Boyd schrieb: > Johannes Sixt wrote: >> OTOH, you can simply revert the hunk from 518ef8f (completion: Replace >> config --list with --get-regexp, 2009-09-11) that applies here, or even >> only reinstate the case statement in the loop body without changing >> anything else. > > I'm not opposed to the idea, but then it wouldn't work with something like > > [alias] > whowhat = "log --format="%an\nalias.broken=foo'" > > although that actually happening is probably rare, it can still happen. I'd ignore this case, but *you* write the patch, *you* get to draw the line ;) -- Hannes