Dirk Mueller
878c4bc78f
Suppress GCC 6 warning about ambiguous 'else' with -Wparentheses OBS-URL: https://build.opensuse.org/request/show/391528 OBS-URL: https://build.opensuse.org/package/show/Base:System/glibc?expand=0&rev=433
89 lines
2.3 KiB
Diff
89 lines
2.3 KiB
Diff
From: Yvan Roux <yvan.roux@linaro.org>
|
|
Date: Fri, 15 Apr 2016 13:29:26 +0200
|
|
Subject: Suppress GCC 6 warning about ambiguous 'else' with -Wparentheses
|
|
Patch-mainline: yes
|
|
Git-commit: df1cf48777fe4cd81ad7fb09ecbe5b31432b7c1c
|
|
|
|
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
|
|
---
|
|
ChangeLog | 5 +++++
|
|
nis/nis_call.c | 20 +++++++++++---------
|
|
stdlib/setenv.c | 24 +++++++++++++-----------
|
|
3 files changed, 29 insertions(+), 20 deletions(-)
|
|
|
|
--- a/ChangeLog
|
|
+++ b/ChangeLog
|
|
@@ -1,3 +1,8 @@
|
|
+2016-04-15 Yvan Roux <yvan.roux@linaro.org>
|
|
+
|
|
+ * stdlib/setenv.c (unsetenv): Fix ambiguous 'else'.
|
|
+ * nis/nis_call.c (nis_server_cache_add): Likewise.
|
|
+
|
|
2016-02-18 Adhemerval Zanella <adhemerval.zanella@linaro.org>
|
|
|
|
* version.h (RELEASE): Set to "stable".
|
|
--- a/nis/nis_call.c
|
|
+++ b/nis/nis_call.c
|
|
@@ -680,16 +680,18 @@ nis_server_cache_add (const_nis_name nam
|
|
/* 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;
|
|
|
|
--- a/stdlib/setenv.c
|
|
+++ b/stdlib/setenv.c
|
|
@@ -278,18 +278,20 @@ unsetenv (const char *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;
|
|
+ {
|
|
+ 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;
|
|
+ do
|
|
+ dp[0] = dp[1];
|
|
+ while (*dp++);
|
|
+ /* Continue the loop in case NAME appears again. */
|
|
+ }
|
|
+ else
|
|
+ ++ep;
|
|
+ }
|
|
|
|
UNLOCK;
|
|
|