Tom Lane писал 2021-07-26 18:18:
> Alexander Pyhalov <a.pyhalov@postgrespro.ru> writes:
>> [ 0001-Allow-pushing-CASE-expression-to-foreign-server-v6.patch ]
>
> This doesn't compile cleanly:
>
> deparse.c: In function 'foreign_expr_walker.isra.4':
> deparse.c:920:8: warning: 'collation' may be used uninitialized in
> this function [-Wmaybe-uninitialized]
> if (collation != outer_cxt->collation)
> ^
> deparse.c:914:3: warning: 'state' may be used uninitialized in this
> function [-Wmaybe-uninitialized]
> switch (state)
> ^~~~~~
>
> These uninitialized variables very likely explain the fact that it
> fails
> regression tests, both for me and for the cfbot. Even if this weren't
> an
> outright bug, we don't tolerate code that produces warnings on common
> compilers.
>
> regards, tom lane
Hi.
Of course, this is a patch issue. Don't understand how I overlooked
this.
Rebased on master and fixed it. Tests are passing here (but they also
passed for previous patch version).
What exact tests are failing?
--
Best regards,
Alexander Pyhalov,
Postgres Professional
