Re: [PATCH] fix for C4141 warning on MSVC - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: [PATCH] fix for C4141 warning on MSVC
Date
Msg-id CAEepm=1yQZ1kM89=JVkJRL_1_c6uh1v5HkuvB=j0g7NAU1tCiQ@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] fix for C4141 warning on MSVC  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCH] fix for C4141 warning on MSVC
List pgsql-hackers
On Wed, Jan 24, 2018 at 1:42 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Thomas Munro <thomas.munro@enterprisedb.com> writes:
>> On Wed, Jan 24, 2018 at 1:16 PM, Michail Nikolaev
>> <michail.nikolaev@gmail.com> wrote:
>>> Just very small fix for C4141 warning
>
>> Thanks.  This is similar to the fix I proposed over here:
>> https://www.postgresql.org/message-id/CAEepm%3D2iTKvbebiK3CdoczQk4_FfDt1EeU4c%2BnGE340JH7gQ0g%40mail.gmail.com
>
>> Perhaps we should combine these?
>
> +1.  The previous thread seems to have died off with nothing happening,
> but we still have those issues to resolve.  I like the idea of making
> the macro be a direct drop-in substitute for "inline" rather than
> something you use in addition to "inline".  And if we do that, it
> definitely should expand to plain "inline" if we have no other idea.

Here's one like that.

-- 
Thomas Munro
http://www.enterprisedb.com

Attachment

pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Possible performance regression in version 10.1 with pgbenchread-write tests.
Next
From: Michael Paquier
Date:
Subject: Re: PATCH: Configurable file mode mask