Re: Compiler warning when using TRGM_REGEXP_DEBUG

Sergei Kornilov <sk@zsrv.org> writes:
> I was looking into how pg_trgm index support works with regular expressions. When building HEAD with the
TRGM_REGEXP_DEBUGoption, my compiler warns: 

> trgm_regexp.c: In function ‘printSourceNFA’:
> trgm_regexp.c:2142:41: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local]
>  2142 |                 int                     i,
>       |                                         ^
> trgm_regexp.c:2133:33: note: shadowed declaration is here
>  2133 |         int                     i;

Ah, yeah, I see it too with -DTRGM_REGEXP_DEBUG.

> Does it make sense to fix this warning? It is possible to either rename this variable or remove the second
declaration(the variable is used in another loop after this code). 

I think if we need to touch it, we should modernize all the loops in
this function to use loop-local variables.  Will see to it.

            regards, tom lane