[Asterisk-code-review] BuildSystem: For consistency, avoid double-checking via if c... (asterisk[master])

Corey Farrell asteriskteam at digium.com
Sun Mar 11 13:49:56 CDT 2018


Corey Farrell has posted comments on this change. ( https://gerrit.asterisk.org/8492 )

Change subject: BuildSystem: For consistency, avoid double-checking via if clauses.
......................................................................


Patch Set 1: Code-Review+1

I assume checking switching the symbol being checked is just for consistency with the other checks.  This is fine since both symbols are used by main/backtrace.c.


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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4a6f48127b7af3a48168c917e888be1f70625027
Gerrit-Change-Number: 8492
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Sun, 11 Mar 2018 18:49:56 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180311/c856dfbe/attachment.html>


More information about the asterisk-code-review mailing list