[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Accepting [xyz---abc] - three minus signs to mean one

From: arnold
Subject: Re: Accepting [xyz---abc] - three minus signs to mean one
Date: Sun, 24 Apr 2022 07:21:06 -0600
User-agent: Heirloom mailx 12.5 7/5/10

Hi Paul.

Thanks for this. The patch looks good. I will (eventually) merge it
into gawk instead of my change.

I plan to add a test to gawk; perhaps grep would benefit from one as well?



Paul Eggert <eggert@cs.ucla.edu> wrote:

> On 4/21/22 00:57, Arnold Robbins wrote:
> > As far as my testing indicates, dfa.c doesn't need a patch, it seems
> > to accept "---" inside brackets for a single minus.
> Yes, a brief perusal of the dfa.c source code suggests you're right. 
> Thanks for looking into this. I tend to agree with you that POSIX is not 
> likely to outlaw this extension.
> > If there are no objections, can we get this into Gnulib?
> Although the basic idea looks good, I see a few places where the patch 
> can be improved.
> * The two calls to re_string_peek_byte might go past the end of the 
> pattern (a subscript violation). This is possible because the pattern is 
> not necessarily null-terminated.
> * The two calls to re_string_fetch_byte can be simplified into a single 
> call to re_string_skip_bytes.
> * No need to assign to token->opr.c, as it already has the correct value.
> * Can fall through to the default case to save a bit of duplicate code.
> * glibc still uses comments /* like this */ for style reasons, and we 
> should stick to that.
> I wrote a patch with these improvements in mind and installed it into 
> Gnulib (see attached); hope it works for Gawk too.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]