Frank Quinn created PROTON-419:

             Summary: Warnings Presented When Using Strict Compiler Flags
                 Key: PROTON-419
             Project: Qpid Proton
          Issue Type: Bug
          Components: proton-c
    Affects Versions: 0.5
         Environment: GCC 4.4.7
            Reporter: Frank Quinn

Hi Folks,

In our project, we currently use the following compiler flags for gcc (tested 
in version 4.4.7):

-Wall -Wdeclaration-after-statement -Wmissing-prototypes -Wstrict-prototypes 

Unfortunately this causes problems in recent versions of proton as its headers 
produce some warnings of their own. The ones we have spotted can be recreated 
using the following code:

#include <proton/error.h>
#include <proton/messenger.h>

int main(){}

Compiled with:

$ gcc -Wall -Wdeclaration-after-statement -Wmissing-prototypes 
-Wstrict-prototypes -Werror -I./qpid-proton-0.5/proton-c/include/ test.c -o test
cc1: warnings being treated as errors
In file included from test.c:1:
./qpid-proton-0.5/proton-c/include/proton/error.h:46: error: function 
declaration isn’t a prototype
In file included from test.c:2:
./qpid-proton-0.5/proton-c/include/proton/messenger.h:458:29: error: "/*" 
within comment
./qpid-proton-0.5/proton-c/include/proton/messenger.h:466:40: error: "/*" 
within comment
./qpid-proton-0.5/proton-c/include/proton/messenger.h:470:32: error: "/*" 
within comment
./qpid-proton-0.5/proton-c/include/proton/messenger.h:475:34: error: "/*" 
within comment
test.c:4: error: function declaration isn’t a prototype

The first one can be resolved by changing pn_error() to pn_error(void) and the 
others can be fixed simply by adding some spacing. We typically just hack our 
local copies for testing, but we figured it may be better to get this into the 
main code stream if you guys can find the time to put it in.


This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

Reply via email to