Re: Compiler warning when using TRGM_REGEXP_DEBUG

Sergei Kornilov <sk(at)zsrv(dot)org> writes:
> I was looking into how pg_trgm index support works with regular expressions. When building HEAD with the TRGM_REGEXP_DEBUG option, 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