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

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


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


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/04/8104/1

diff --git a/main/backtrace.c b/main/backtrace.c
index aed4ffd..384c231 100644
--- a/main/backtrace.c
+++ b/main/backtrace.c
@@ -130,7 +130,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/8104
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ec3e144d425a976c02811bd23cd0c533d2eca4e
Gerrit-Change-Number: 8104
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/dd803a81/attachment-0001.html>


More information about the asterisk-code-review mailing list