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

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


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


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/06/8106/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/8106
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9ec3e144d425a976c02811bd23cd0c533d2eca4e
Gerrit-Change-Number: 8106
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/e29de10b/attachment.html>


More information about the asterisk-code-review mailing list