diff options
| author | Tom Lane | 2019-06-02 15:07:54 +0000 |
|---|---|---|
| committer | Tom Lane | 2019-06-02 15:07:54 +0000 |
| commit | 10a53cae9993abae159abdc6a76e46f6aac120d1 (patch) | |
| tree | c7b9b80b9f4a2aa11a74454eb5123f16324b9cbe /src/tools/pginclude/cpluspluscheck | |
| parent | 4b3f1dd71b74f82dedbf37c331d0a444368c4009 (diff) | |
Un-break ecpg tests for Windows.
Declaring a function "inline" still doesn't work with Windows compilers
(C99? what's that?), unless the macro provided by pg_config.h is
in-scope, which it is not in our ECPG test programs. So the workaround
I tried to use in commit 7640f9312 doesn't work for Windows. Revert
the change in printf_hack.h, and instead just blacklist that file
in cpluspluscheck --- since it's a not-installed test file, we don't
really need to verify its C++ cleanliness anyway.
Diffstat (limited to 'src/tools/pginclude/cpluspluscheck')
| -rwxr-xr-x | src/tools/pginclude/cpluspluscheck | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/tools/pginclude/cpluspluscheck b/src/tools/pginclude/cpluspluscheck index 38617843665..b3fc75de48a 100755 --- a/src/tools/pginclude/cpluspluscheck +++ b/src/tools/pginclude/cpluspluscheck @@ -105,6 +105,8 @@ do # regression.h is not actually C, but ECPG code. test "$f" = src/interfaces/ecpg/test/regression.h && continue + # printf_hack.h produces "unused function" warnings. + test "$f" = src/interfaces/ecpg/test/printf_hack.h && continue # pg_trace.h and utils/probes.h can include sys/sdt.h from SystemTap, # which itself contains C++ code and so won't compile with a C++ |