Suppress GCC 6 warning about ambiguous 'else' with -Wparentheses

Backport of df1cf48777fe4cd81ad7fb09ecbe5b31432b7c1c.

	* stdlib/setenv.c (unsetenv): Fix ambiguous 'else'.
	* nis/nis_call.c (nis_server_cache_add): Likewise.

diff --git a/ChangeLog b/ChangeLog
index b36cd9e17a0532c77cb716d2dbb6ea692069eabf..666be708d2e6469830740375fe14f6d363d5c210 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2016-05-09  Yvan Roux  <yvan.roux@linaro.org>
+
+       * stdlib/setenv.c (unsetenv): Fix ambiguous 'else'.
+       * nis/nis_call.c (nis_server_cache_add): Likewise.
+
 2016-04-28  Stefan Liebler  <stli@linux.vnet.ibm.com>
 
        [BZ #18508]

diff --git a/nis/nis_call.c b/nis/nis_call.c
index 970415b505663fceb14cf4cd6eeb198be88518ed..d98c3859f45fc1c0be6b48b089b9822d4f18362a 100644 (file)
--- a/nis/nis_call.c
+++ b/nis/nis_call.c
@@ -680,16 +680,18 @@ nis_server_cache_add (const_nis_name name, int search_parent,
   /* Choose which entry should be evicted from the cache.  */
   loc = &nis_server_cache[0];
   if (*loc != NULL)
-    for (i = 1; i < 16; ++i)
-      if (nis_server_cache[i] == NULL)
-       {
+    {
+      for (i = 1; i < 16; ++i)
+       if (nis_server_cache[i] == NULL)
+         {
+           loc = &nis_server_cache[i];
+           break;
+         }
+       else if ((*loc)->uses > nis_server_cache[i]->uses
+                || ((*loc)->uses == nis_server_cache[i]->uses
+                    && (*loc)->expires > nis_server_cache[i]->expires))
          loc = &nis_server_cache[i];
-         break;
-       }
-      else if ((*loc)->uses > nis_server_cache[i]->uses
-              || ((*loc)->uses == nis_server_cache[i]->uses
-                  && (*loc)->expires > nis_server_cache[i]->expires))
-       loc = &nis_server_cache[i];
+    }
   old = *loc;
   *loc = new;
 

diff --git a/stdlib/setenv.c b/stdlib/setenv.c
index b60c4f0151a623669b5cdb738528cac4e0a15643..84cec794fbdbdfe6bbf9077f8001280618ed491e 100644 (file)
--- a/stdlib/setenv.c
+++ b/stdlib/setenv.c
@@ -269,18 +269,20 @@ unsetenv (name)
   ep = __environ;
   if (ep != NULL)
     while (*ep != NULL)
-      if (!strncmp (*ep, name, len) && (*ep)[len] == '=')
-       {
-         /* Found it.  Remove this pointer by moving later ones back.  */
-         char **dp = ep;
-
-         do
-           dp[0] = dp[1];
-         while (*dp++);
-         /* Continue the loop in case NAME appears again.  */
-       }
-      else
-       ++ep;
+      {
+       if (!strncmp (*ep, name, len) && (*ep)[len] == '=')
+         {
+           /* Found it.  Remove this pointer by moving later ones back.  */
+           char **dp = ep;
+
+           do
+               dp[0] = dp[1];
+           while (*dp++);
+           /* Continue the loop in case NAME appears again.  */
+         }
+       else
+         ++ep;
+      }
 
   UNLOCK;