Ticket #161 (closed defect: wontfix)

Opened 7 years ago

Last modified 6 years ago

Request to add CFLAG -Werror=declaration-after-statement

Reported by:	jmckenna	Owned by:
Priority:	major	Milestone:
Component:	Development platform	Version:
Keywords:		Cc:	jmckenna@…

Description

The C89 requirement is a wall that is hit by many software projects (PROJ.4, GDAL, MapServer/MapCache?) as many different compilers support different standards.

MapServer/MapCache? recently added the -Werror=declaration-after-statement to avoid compiler errors when variables are not declared at the beginning of the block: https://github.com/mapserver/mapcache/commit/219e54b45e4730b91111a6874e435a8ca8a1aa6d

Even Rouault recently mentioned that PROJ.4 also uses that flag, which is based on the GDAL use of it also: https://github.com/OSGeo/proj.4/blob/master/m4/ax_cflags_warn_all.m4#L35

Here are more ZOO-Project compile errors on Windows:

d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(273) : error C2061: syntax error : identifier 'hasKey'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(273) : error C2059: syntax error : ';'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(273) : error C2059: syntax error : 'type'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(285) : error C2061: syntax error : identifier 'hasElement'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(285) : error C2059: syntax error : ';'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(285) : error C2059: syntax error : 'type'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(297) : error C2061: syntax error : identifier 'contains'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(297) : error C2059: syntax error : ';'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(297) : error C2059: syntax error : 'type'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(316) : error C2061: syntax error : identifier 'addServiceToRegistry'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(316) : error C2059: syntax error : ';'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(316) : error C2059: syntax error : 'type'
d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(328) : warning C4273: '_snprintf' : inconsistent dll linkage
        C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC\INCLUDE\stdio.h(349) : see previous definition of '_snprintf'
ulinet.c(92) : error C2143: syntax error : missing ';' before 'type'
ulinet.c(93) : error C2275: '_HINTERNET' : illegal use of this type as an expression
        d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\ulinet.h(87) : see declaration of '_HINTERNET'
ulinet.c(93) : error C2065: 'psInternet' : undeclared identifier
ulinet.c(94) : error C2065: 'psInternet' : undeclared identifier
ulinet.c(94) : warning C4047: '!=' : 'int' differs in levels of indirection from 'void *'
ulinet.c(95) : error C2065: 'psInternet' : undeclared identifier
ulinet.c(95) : error C2223: left of '->cookie' must point to struct/union
ulinet.c(96) : error C2065: 'psInternet' : undeclared identifier
ulinet.c(96) : error C2223: left of '->cookie' must point to struct/union
ulinet.c(246) : warning C4013: 'strtok_r' undefined; assuming extern returning int
ulinet.c(246) : warning C4047: '=' : 'char *' differs in levels of indirection from 'int'
ulinet.c(247) : error C2143: syntax error : missing ';' before 'type'
ulinet.c(248) : error C2143: syntax error : missing ';' before 'type'
ulinet.c(249) : error C2065: 'cnt' : undeclared identifier
ulinet.c(251) : error C2065: 'cnt' : undeclared identifier
ulinet.c(254) : error C2065: 'cnt' : undeclared identifier
ulinet.c(258) : warning C4047: '=' : 'char *' differs in levels of indirection from 'int'
ulinet.c(259) : error C2065: 'cnt' : undeclared identifier
ulinet.c(281) : warning C4047: '=' : 'char *' differs in levels of indirection from 'int'
ulinet.c(282) : error C2143: syntax error : missing ';' before 'type'
ulinet.c(283) : error C2065: 'i' : undeclared identifier
ulinet.c(284) : error C2065: 'i' : undeclared identifier
ulinet.c(289) : error C2143: syntax error : missing ';' before 'type'
ulinet.c(290) : error C2065: 'j' : undeclared identifier
ulinet.c(291) : error C2065: 'j' : undeclared identifier
ulinet.c(292) : error C2065: 'j' : undeclared identifier
ulinet.c(292) : error C2065: 'i' : undeclared identifier
ulinet.c(294) : error C2065: 'j' : undeclared identifier
ulinet.c(295) : error C2065: 'j' : undeclared identifier
ulinet.c(298) : error C2275: 'map' : illegal use of this type as an expression
        d:\build\mapserver-buildkit-2012\zoo-project-svn-20170731\zoo-project\zoo-kernel\service.h(191) : see declaration of 'map'
ulinet.c(298) : error C2065: 'tmpMap' : undeclared identifier
ulinet.c(299) : error C2065: 'tmpMap' : undeclared identifier
ulinet.c(299) : warning C4047: '!=' : 'int' differs in levels of indirection from 'void *'
ulinet.c(300) : error C2065: 'i' : undeclared identifier
ulinet.c(300) : error C2065: 'tmpMap' : undeclared identifier
ulinet.c(300) : error C2223: left of '->value' must point to struct/union
ulinet.c(300) : error C2198: 'AddMissingHeaderEntry' : too few arguments for call
ulinet.c(302) : error C2065: 'i' : undeclared identifier
ulinet.c(303) : error C2065: 'i' : undeclared identifier
ulinet.c(305) : warning C4047: '=' : 'char *' differs in levels of indirection from 'int'
ulinet.c(539) : warning C4018: '<' : signed/unsigned mismatch
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC\BIN\cl.EXE"' : return code '0x2'
Stop.

Change History

Changed 7 years ago by djay

Ok, I understand this requirement.

What I don't understand here is that, I suppose you have tried using this flag but it did not solve the issue completely. Can we know what are the compilation result after adding this flag ?

I have another question, did you try to build the 1.6.0 release on windows platform ? I ask this question cause it was the last one that has been tested and build on windows platform. So I suppose that in case the 1.6.0 build then we will "only" to have to track changes and check what I can break on the way. The work to have the windows build working was handled by Knut Landmark, in case you see this ticket, I think your help would be much appreciated here.

I hope we can give a try to the 1.6.0 release to be built on windows.

Changed 7 years ago by jmckenna

cc jmckenna@… added

I apologize for not explaining more. This ticket is in fact requesting that the CFLAG switch -Werror=declaration-after-statement be added to the ZOO-Project's main configure file (or configure.ac) - therefore that switch cannot be tested on Windows.

As mentioned previously, this switch handles the requirement of some compilers that follow the C89 standard, where all variables must be declared/initialized at the beginning of a code block - directly after an opening brace { Most OSGeo projects handle that requirement by including the -Werror=declaration-after-statement in their configure script, such as:

MapServer: https://github.com/mapserver/mapserver/blob/branch-7-0/CMakeLists.txt#L169

GDAL: configure line#3942 for ac_arg in "-pedantic -Wdeclaration-after-statement % -Wall -Wdeclaration-after-statement" "-pedantic % -Wall"

MapCache: https://github.com/mapserver/mapcache/commit/219e54b45e4730b91111a6874e435a8ca8a1aa6d

PROJ.4: https://github.com/OSGeo/proj.4/blob/master/m4/ax_cflags_warn_all.m4#L35

Once that switch is added, then the next time you or any other developer tries to compile the ZOO-Project from source, on Ubuntu or whatever Unix flavor, there will be many errors thrown, showing these C89 variable problems (you could test this now as you read, on your machine there). Once those are fixed and committed, then the next time a developer tries to add some C code, the C89 rules are verified in their new code, by that -Wdeclaration-after-statement switch.

Please let me know if that explanation helps, for describing the purpose of this ticket.

Regarding the 1.6.0 release and Windows:

The MS4W build environment uses the Microsoft Visual Studio 2012 compiler (because that specific compiler is required for PHP 5.6, and Python 2.x). The initial ZOO-Project windows builds were done on the Visual Studio 2013 compiler in appveyor by Knut Landmark - the 2013 compiler is more forgiving, and these C89 requirements don't exist.

Of course I can use the 1.6.0 release (yes I have tried, and the same C89 errors, where variables are not declared at the beginning of the block, exist).

So in summary: this will be a team effort. Can you try adding that CFLAG switch to configure in your environment? I will also continue to see if I can fix the C89 errors (hopefully once you add that switch and try compiling, you can also help with the errors).

Thanks. -jeff

Note: See TracTickets for help on using tickets.

Download in other formats:

Open WPS platform

Ticket #161 (closed defect: wontfix)

Request to add CFLAG -Werror=declaration-after-statement

Description

Change History

Changed 7 years ago by djay

Changed 7 years ago by jmckenna

Changed 7 years ago by djay

Changed 7 years ago by jmckenna

Changed 7 years ago by jmckenna

Changed 7 years ago by djay

Changed 6 years ago by jmckenna

Download in other formats:

Download now !

Support ZOO-Project

Search

Context Navigation

Main Navigation

ZOO Sponsors

Knowledge partners

Related links