[Asterisk-code-review] bridge softmix: Fix sfu append source streams test. (asterisk[master])

Corey Farrell asteriskteam at digium.com
Mon Jan 8 20:28:04 CST 2018


Hello Richard Mudgett, Jenkins2, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/7876

to look at the new patch set (#2).

Change subject: bridge_softmix: Fix sfu_append_source_streams test.
......................................................................

bridge_softmix: Fix sfu_append_source_streams test.

* validate_stream: zero result from ast_format_cap_identical indicates
  they are not identical, rather than non-zero indicating an error.
* validate_original_streams: use num_streams instead of
  ARRAY_LEN(params).
* Fix declaration of alice_dest_stream and bob_dest_stream.

Change-Id: I6b1dd8bed10439d3c7406f033eb1896b6c419147
---
M bridges/bridge_softmix.c
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/76/7876/2
-- 
To view, visit https://gerrit.asterisk.org/7876
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6b1dd8bed10439d3c7406f033eb1896b6c419147
Gerrit-Change-Number: 7876
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180108/f16d4551/attachment.html>


More information about the asterisk-code-review mailing list