Re: Windows build warnings - Mailing list pgsql-hackers

From Juan José Santamaría Flecha
Subject Re: Windows build warnings
Date
Msg-id CAC+AXB3drqaU3aT-NwFjMatbFhiiXGVOYWvjdTRPO9aW_p3j1Q@mail.gmail.com
Whole thread Raw
In response to Re: Windows build warnings  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Windows build warnings
List pgsql-hackers

On Tue, Nov 23, 2021 at 2:11 PM Daniel Gustafsson <daniel@yesql.se> wrote:
> On 22 Nov 2021, at 16:40, Tom Lane <tgl@sss.pgh.pa.us> wrote:

> I can't find anything that is providing a non-empty definition of
> PG_USED_FOR_ASSERTS_ONLY (a/k/a pg_attribute_unused) for anything
> except GCC. 

It's supported in clang as well per the documentation [0] in at least some
configurations or distributions:

        "The [[maybe_unused]] (or __attribute__((unused))) attribute can be
        used to silence such diagnostics when the entity cannot be removed.
        For instance, a local variable may exist solely for use in an assert()
        statement, which makes the local variable unused when NDEBUG is
        defined."

[[maybe_unused]] is also recognized from Visual Studio 2017 onwards [1].


Regards,

Juan José Santamaría Flecha

pgsql-hackers by date:

Previous
From: Amul Sul
Date:
Subject: Re: [Patch] ALTER SYSTEM READ ONLY
Next
From: Daniel Gustafsson
Date:
Subject: Re: Support for NSS as a libpq TLS backend