pgsql: configure: Apply -Werror=vla to C++ as well as C - Mailing list pgsql-committers

From Peter Eisentraut
Subject pgsql: configure: Apply -Werror=vla to C++ as well as C
Date
Msg-id E1w7Hly-002BtZ-0n@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
configure: Apply -Werror=vla to C++ as well as C

The comment part of d9dd406fe281 mentioned that -Wvla is not applicable
for C++. That is not fully correct: it is true that VLAs are not part of the
C++ standard, and g++ with -pedantic will also warn about them as a non-standard
extension.  However, -Wvla itself works fine in C++ and will catch VLA
usage just as in C.

Fix configure.ac to apply -Werror=vla to C++ as well. There is no need to
fix meson.build as it already includes it in common_warning_flags.

Author: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Suggested-by: Peter Eisentraut <peter@eisentraut.org>
Discussion: https://postgr.es/m/7bf60ab1-2b5d-4a77-93ce-815072a0a014%40eisentraut.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/488ab592d9ea1b4c7d6e43389c34c4bbbcb8f0d4

Modified Files
--------------
configure    | 54 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
configure.ac |  4 ++--
2 files changed, 54 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Be more careful to preserve consistency of a tuplestore.
Next
From: Nathan Bossart
Date:
Subject: pgsql: Add commit 874da8b1f6 to .git-blame-ignore-revs.