[Asterisk-code-review] res rtp asterisk: rtp->ice check not wrapped in USE PJPROJEC... (asterisk[11])

[Richard Mudgett]

Richard Mudgett has posted comments on this change.

Change subject: res_rtp_asterisk: rtp->ice check not wrapped in USE_PJPROJECT ifdef
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/1850/1/res/res_rtp_asterisk.c
File res/res_rtp_asterisk.c:

Line 4722: #ifdef USE_PJPROJECT
         : 	if (rtp->ice) {
         : 		return;
         : 	}
         : #endif
Move this check inside the #ifdef HAVE_OPENSSL_SRTP conditional area.  No sense in having this check if we also don't HAVE_OPENSSL_SRTP


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I19b49112e1b630bd04e859f14ccf96f8ebd6b151
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes