[Asterisk-code-review] pjproject bundled: Compile pjsua with max log level = 2 (asterisk[13])

Richard Mudgett asteriskteam at digium.com
Tue Jan 3 17:01:57 CST 2017


Richard Mudgett has posted comments on this change. ( https://gerrit.asterisk.org/4691 )

Change subject: pjproject_bundled:  Compile pjsua with max log level = 2
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.asterisk.org/#/c/4691/1/third-party/pjproject/patches/config_site.h
File third-party/pjproject/patches/config_site.h:

Line 25: #ifndef NDEBUG
> Why "doesn't it matter"?
The defined is only checked if it exists or not.

#ifdef NDEBUG

or

#ifndef NDEBUG

not

#if NDEBUG


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2af9e7d48dde1927279c586c9c725d868fe6f3ff
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list