Fix declaration after statement for C90

tiran · tiran · commit c4f559ca36c4 · 2017-11-29T15:05:36.000+01:00
In strict ISO C90 mode and on some platforms (e.g. older MSVC), it is
not allowed to declare new variables after a statement. I also find it
good practice to declare first.

Fixes error: ISO C90 forbids mixed declarations and code
[-Werror=declaration-after-statement]

Signed-off-by: Christian Heimes &lt;cheimes@redhat.com&gt;
diff --git a/.travis.yml b/.travis.yml
@@ -37,6 +37,13 @@ matrix:
     - python: 3.6
       env: TOXENV=doc
 
+env:
+    global:
+        # -Werror=declaration-after-statement: strict ISO C90
+        - CFLAGS="-std=c90 -Werror=declaration-after-statement"
+        # pass CFLAGS and WITH_GCOV to tox tasks
+        - TOX_TESTENV_PASSENV="CFLAGS WITH_GCOV"
+
 install:
   - pip install "pip>=7.1.0"
   - pip install tox-travis tox codecov coverage
diff --git a/Modules/LDAPObject.c b/Modules/LDAPObject.c
@@ -999,6 +999,9 @@ l_ldap_result4( LDAPObject* self, PyObject *args )
     int res_msgid = 0;
     char *retoid = 0;
     PyObject *valuestr = NULL;
+    int result = LDAP_SUCCESS;
+    char **refs = NULL;
+    LDAPControl **serverctrls = 0;
 
     if (!PyArg_ParseTuple( args, "|iidiii", &msgid, &all, &timeout, &add_ctrls, &add_intermediates, &add_extop ))
         return NULL;
@@ -1034,9 +1037,6 @@ l_ldap_result4( LDAPObject* self, PyObject *args )
     if (msg)
         res_msgid = ldap_msgid(msg);
 
-    int result = LDAP_SUCCESS;
-    char **refs = NULL;
-    LDAPControl **serverctrls = 0;
     if (res_type == LDAP_RES_SEARCH_ENTRY) {
         /* LDAPmessage_to_python will parse entries and read the controls for each entry */
     } else if (res_type == LDAP_RES_SEARCH_REFERENCE) {
diff --git a/Modules/constants.c b/Modules/constants.c
@@ -60,12 +60,11 @@ LDAPerror( LDAP *l, char *msg )
     PyObject *info;
     PyObject *str;
     PyObject *pyerrno;
+    char *matched, *error;
 
     /* at first save errno for later use before it gets overwritten by another call */
     myerrno = errno;
 
-    char *matched, *error;
-
     opt_errnum = ldap_get_option(l, LDAP_OPT_ERROR_NUMBER, &errnum);
     if (opt_errnum != LDAP_OPT_SUCCESS)
       errnum = opt_errnum;
diff --git a/Modules/message.c b/Modules/message.c
@@ -92,10 +92,11 @@ LDAPmessage_to_python(LDAP *ld, LDAPMessage *m, int add_ctrls, int add_intermedi
 	 ) {
 	     PyObject* valuelist;
 	     PyObject* pyattr;
+	     struct berval **bvals;
+
 	     pyattr = PyUnicode_FromString(attr);
 
-	     struct berval ** bvals =
-	     	ldap_get_values_len( ld, entry, attr );
+	     bvals = ldap_get_values_len( ld, entry, attr );
 
 	     /* Find which list to append to */
 	     if ( PyDict_Contains( attrdict, pyattr ) ) {
