-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3379/
-----------------------------------------------------------
(Updated March 20, 2014, 2:36 p.m.)
Review request for Asterisk Developers.
Changes
-------
Switch to reject.
Bugs: ASTERISK-22677
https://issues.asterisk.org/jira/browse/ASTERISK-22677
Repository: Asterisk
Description
-------
Previously, if you played an audio file and then played another before the
first finished, the second audio file would start playing immediately as it was
called overlapping the previous sound. Apparently people don't like that. This
patch changes that behavior so that the sound will be queued at the end of any
existing controls if they are running.
Diffs (updated)
-----
/branches/12/res/res_stasis_playback.c 410875
/branches/12/res/res_stasis.c 410875
/branches/12/res/ari/resource_bridges.c 410875
/branches/12/include/asterisk/stasis_app.h 410875
Diff: https://reviewboard.asterisk.org/r/3379/diff/
Testing
-------
Tested for playback channel wrapper leaks, tested to make sure control objects
were being destroyed when they fell out of use. Tested playing of a single
file. Tested playing of multiple files in a row. Tested playing of multiple
files in a row and then after a sequence finished, playing additional files so
that new channels would have to be created. Tested playing sounds right as
other sounds were concluding. I wasn't able to break it (although I wouldn't be
surprised if there is a possible condition where you can grab a control as it
is finishing up its queue and then attempting to add a sound to a finished
queue causing the playback to fail. I don't think this would break things in a
profound way, it just might possibly make one sound fail to queue under
extremely unlikely conditions).
Thanks,
Jonathan Rose
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
