Re: GNU poke 2.0.91 pre-released in alpha.gnu.org

[Bruno Haible]

Jose E. Marchesi wrote:
> > regexec takes nmatch, pmatch arguments. If nmatch is 1, pmatch must be an
> > array of length 1, not NULL.
> 
> regex(3) says:
> 
>        REG_NOSUB
>               Do  not report position of matches.  The nmatch and pmatch argu‐
>               ments to regexec() are ignored if the  pattern  buffer  supplied
>               was compiled with this flag set.
> 
> In this case we used REG_NOSUB to compile the pattern buffer.

Indeed, POSIX 
<https://pubs.opengroup.org/onlinepubs/009696899/functions/regcomp.html>
too says:
       If nmatch is 0 or REG_NOSUB was set in the cflags argument to regcomp(),
       then regexec() shall ignore the pmatch argument.

So, the combination of glibc headers and GCC causes a false warning. But I
guess that it will stay, because for the other cases (without REG_NOSUB) it
is a useful warning.

> I committed the patch below anyway.

Should work; thanks. Another possibility would have been to pass nmatch = 0
instead of nmatch = 1.

Bruno