[Asterisk-code-review] bridge softmix.c: Remove always true test. (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Tue Aug 22 11:11:26 CDT 2017


Richard Mudgett has uploaded this change for review. ( https://gerrit.asterisk.org/6261


Change subject: bridge_softmix.c: Remove always true test.
......................................................................

bridge_softmix.c: Remove always true test.

Change-Id: I26238df2ff0d0f6dfe95c3aa35da588f1ee71727
---
M bridges/bridge_softmix.c
1 file changed, 2 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/61/6261/1

diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c
index b359948..b9a0e98 100644
--- a/bridges/bridge_softmix.c
+++ b/bridges/bridge_softmix.c
@@ -1060,14 +1060,10 @@
 		res = ast_bridge_queue_everyone_else(bridge, bridge_channel, frame);
 		break;
 	case AST_FRAME_VOICE:
-		if (bridge_channel) {
-			softmix_bridge_write_voice(bridge, bridge_channel, frame);
-		}
+		softmix_bridge_write_voice(bridge, bridge_channel, frame);
 		break;
 	case AST_FRAME_VIDEO:
-		if (bridge_channel) {
-			softmix_bridge_write_video(bridge, bridge_channel, frame);
-		}
+		softmix_bridge_write_video(bridge, bridge_channel, frame);
 		break;
 	case AST_FRAME_CONTROL:
 		res = softmix_bridge_write_control(bridge, bridge_channel, frame);

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I26238df2ff0d0f6dfe95c3aa35da588f1ee71727
Gerrit-Change-Number: 6261
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170822/86fe4674/attachment.html>


More information about the asterisk-code-review mailing list