[asterisk-dev] [Code Review] 3910: Fix test failures introduced by 420934

[Jonathan Rose]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3910/
-----------------------------------------------------------

(Updated Aug. 15, 2014, 11:56 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, Matt Jordan and rmudgett.


Changes
-------

Committed in revision 421186


Bugs: ASTERISK-24027
    https://issues.asterisk.org/jira/browse/ASTERISK-24027


Repository: Asterisk


Description
-------

r420934 changed ast_channel_is_leaving bridge to not care about the unbridge flag (which is no longer a softhangup flag due to that causing the behavior noted in ASTERISK-24027). This patch makes it so that the unbridge flag is evaluated again since that caused an unexpected change in how media would be re-established when a bridge starts to get torn down on hangup.


Diffs
-----

  /branches/12/main/channel.c 420934 

Diff: https://reviewboard.asterisk.org/r/3910/diff/


Testing
-------

Ran the failing tests from https://bamboo.asterisk.org/bamboo/browse/AST-ATSF-C632TE-336.  I couldn't get all of them to pass prior to r420934, but the ones that I could run successfully were confirmed broken by r420934 and fixed by this patch.


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140815/171d64c4/attachment-0001.html>