[Asterisk-code-review] backtrace: Avoid -Wlogical-not-parentheses. (asterisk[13])

Alexander Traud asteriskteam at digium.com
Tue Jan 30 09:59:03 CST 2018


Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/8105


Change subject: backtrace: Avoid -Wlogical-not-parentheses.
......................................................................

backtrace: Avoid -Wlogical-not-parentheses.

clang 4.0 warned about this.

ASTERISK-27642

Change-Id: I9ec3e144d425a976c02811bd23cd0c533d2eca4e
---
M main/backtrace.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/05/8105/1

diff --git a/main/backtrace.c b/main/backtrace.c
index 1659890..263cdbd 100644
--- a/main/backtrace.c
+++ b/main/backtrace.c
@@ -131,7 +131,7 @@
 			}
 
 			for (section = bfdobj->sections; section; section = section->next) {
-				if (!bfd_get_section_flags(bfdobj, section) & SEC_ALLOC ||
+				if ((!bfd_get_section_flags(bfdobj, section)) & SEC_ALLOC ||
 					section->vma > offset ||
 					section->size + section->vma < offset) {
 					continue;

-- 
To view, visit https://gerrit.asterisk.org/8105
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ec3e144d425a976c02811bd23cd0c533d2eca4e
Gerrit-Change-Number: 8105
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180130/9870b72a/attachment.html>


More information about the asterisk-code-review mailing list