Re: major gawk bug

[Aharon Robbins]

Stepan,

RE_ICASE is not the answer. That flag is for use when the regex is
*compiled*.  It would be a performance killer to have to recompile every
regex if IGNORECASE changed since the last time the regex was compiled.

Believe me, if regexec() had a flag to ignore case, I'd be using it.

I think I have a patch which I sent on Stanislav in private email.
If that works, I'll keep it in the tree.

Arnold

> Date: Tue, 8 Jun 2004 14:38:34 +0200
> From: Stepan Kasal <address@hidden>
> To: Aharon Robbins <address@hidden>
> Cc: address@hidden, address@hidden
> Subject: Re: major gawk bug
>
> Hello,
>
> On Tue, Jun 08, 2004 at 03:10:43PM +0300, Aharon Robbins wrote:
> > > (const char casetable[] )
> > > for case-independent regexp matching?
>
> > Code changes welcome.  I have no idea how to do that in a manner that
> > is correct for all 8-bit ASCII-compatible locales.
>
> I beleive the right fix for regexes is to use RE_ICASE flag instead
> of the translate table.
> The hard-coded table is also used in gawk for various case-insensitive
> comparisons; these should be replaced by a call to tolower().
> The hard-coded table should be then removed.
>
> > If you (or someone else) wishes to contribute a patch,
>
> I personally don't have time for it now.  Stanislav, are you willing to
> implement this?
>
> Have a nice day,
>       Stepan