Subject: Re: [PATCH] attr: support quoting pathname patterns in C style

From: Eric Sunshine <[email-masked]>
Date: Thu, 04 Nov 2010 18:53:28 -0400

On 11/4/2010 9:55 AM, Nguyễn Thái Ngọc Duy wrote:
> Full pattern must be quoted. So 'pat"t"ern attr' will give exactly
> 'pat"t"ern', not 'pattern'. Also clarify that leading whitespaces are
> not part of the pattern and document comment syntax.
>
> Signed-off-by: Nguyễn Thái Ngọc Duy<[email-masked]>
> diff --git a/attr.c b/attr.c
> index 6aff695..f3063d8 100644
> --- a/attr.c
> +++ b/attr.c
>   	name = cp;
> -	namelen = strcspn(name, blank);
> +	if (*cp == '"') {
> +		if (unquote_c_style(&pattern, name,&ep))
> +			die("Broken attribute line: %s", line);

The error message is perhaps a bit too generic, indicating only that 
_something_ on the line caused a problem. Mentioning that bad quoting of 
'name' was at fault would help focus the user's attention where it is 
needed.

-- ES