Le 26/07/2021 à 18:03, Alexander Pyhalov a écrit :
Tom Lane писал 2021-07-26 18:18:
Alexander Pyhalov <[email protected]> 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?
I confirm that there is no compilation warning and all regression tests
pass successfully for the v7 patch, I have not checked previous patch
but this one doesn't fail on cfbot too.
Best regards,
--
Gilles Darold
