klibc-bot for Antonio Ospite
2019-Jan-25 03:16 UTC
[klibc] [klibc:update-dash] [PATCH] eval: Silence compiler warning about missing parentheses
Commit-ID: c970d7573aa382c89d3c71b88cd10f4c6a464264
Gitweb:
http://git.kernel.org/?p=libs/klibc/klibc.git;a=commit;h=c970d7573aa382c89d3c71b88cd10f4c6a464264
Author: Antonio Ospite <ao2 at ao2.it>
AuthorDate: Tue, 16 Oct 2018 18:42:20 +0200
Committer: Ben Hutchings <ben at decadent.org.uk>
CommitDate: Fri, 25 Jan 2019 02:57:21 +0000
[klibc] [PATCH] eval: Silence compiler warning about missing parentheses
Gcc gives a warning about some missing parentheses:
-----------------------------------------------------------------------
eval.c: In function ?evaltree?:
eval.c:282:15: warning: logical not is only applied to the left hand side of
comparison [-Wlogical-not-parentheses]
if (!status == isor || evalskip)
^~
eval.c:282:7: note: add parentheses around left hand side expression to silence
this warning
if (!status == isor || evalskip)
^~~~~~~
( )
-----------------------------------------------------------------------
Add the parentheses to silence the warning.
Signed-off-by: Antonio Ospite <ao2 at ao2.it>
Signed-off-by: Herbert Xu <herbert at gondor.apana.org.au>
Signed-off-by: Ben Hutchings <ben at decadent.org.uk>
---
usr/dash/eval.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/usr/dash/eval.c b/usr/dash/eval.c
index 56661880..cecd41c1 100644
--- a/usr/dash/eval.c
+++ b/usr/dash/eval.c
@@ -278,7 +278,7 @@ checkexit:
isor = n->type - NAND;
status = evaltree(n->nbinary.ch1,
(flags | ((isor >> 1) - 1)) & EV_TESTED);
- if (!status == isor || evalskip)
+ if ((!status) == isor || evalskip)
break;
n = n->nbinary.ch2;
evaln:
Reasonably Related Threads
- [klibc:update-dash] dash: eval: Silence compiler warning about missing parentheses
- [klibc:update-dash] eval: Return status in eval functions
- [klibc:update-dash] dash: eval: Return status in eval functions
- [klibc:update-dash] [EVAL] Move common skipcount logic into skiploop
- [klibc:update-dash] dash: [EVAL] Move common skipcount logic into skiploop
