Re: [PATCH 1/3] completion: fix alias listings with newlines

From: Johannes Sixt <j.sixt@viscovery.net>
Date: Oct 8, 2009, 08:10 UTC
Message-ID: <4ACD9E8D.1070702@viscovery.net>
In-Reply-To: <4ACD9C66.1080308@gmail.com>

Stephen Boyd schrieb:

Show 12 quoted lines

> 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