[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] fix C4204 errors while building swig-py with Python 3.9 on Windows

From: Branko Čibej <brane_at_apache.org>
Date: Tue, 17 Nov 2020 11:42:34 +0100

On 17.11.2020 04:09, Jun Omae wrote:
> On Tue, Nov 17, 2020 at 2:33 AM Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> Branko Čibej wrote on Mon, 16 Nov 2020 09:19 +0100:
>>> On 16.11.2020 04:16, Jun Omae wrote:
>>>> The attached patch adds #pragma warning to make C4204 a warning
>>>> instead of an
>>>> error only on building swig-py with Visual Studio.
>>> +1
>> Should that be added to build/generator/templates/ instead? Note that
>> 4204 is specifically there in two places already.
> The C4204 has turned into an error in r1442063.
>
> I've searched reasons of r1442063 in the archive of dev@ mailing list before
> posting the patch, however it was unable to find. I consider that r1442063 is
> to detect breakage of C89/C90 compat, the proposed patch has no changes of
> build/generator/templates/*.ezt files.

The patch is correct. We want to enforce C90 compatibility in our code,
but relax it only when we have to for external code. It makes more sense
to do that in code than in the build generator. There's another example
of this approach in sqlite3wrapper.c.

-- Brane
Received on 2020-11-17 11:42:43 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.