Hi Alvaro,
On Tue, 16 Mar 2021 at 10:20, Alvaro Herrera <alvhe...@alvh.no-ip.org> wrote:
>
> Implement pipeline mode in libpq
> src/test/modules/libpq_pipeline/libpq_pipeline.c | 1303 ++++++++++++++++++++
I'm wondering if you meant to leave the "#define DEBUG" line at line
34 in the above file?
It seems pretty strange to do:
#define DEBUG
#ifdef DEBUG
#define pg_debug(...) do { fprintf(stderr, __VA_ARGS__); } while (0)
#else
#define pg_debug(...)
#endif
pg_debug will never be an empty macro.
I noticed this when testing compiling postgres with MSVC. The compiler
is kicking out a warning: 'DEBUG': macro redefinition, which will be
because that compiler defines DEBUG when doing non-production builds.
David
