[asterisk-dev] [Code Review] app_alarmreceiver cleanup and fixes

KNK reviewboard at asterisk.org
Fri Aug 17 09:54:57 CDT 2012


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

(Updated Aug. 17, 2012, 9:54 a.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed my own findings in the review and some other discovered later.


Summary
-------

Review on behalf of Pedro Kiefer: cleanups from ASTERISK-20224, ASTERISK-20157 and the updated ASTERISK-16694 (author Fred van Lieshout) + improvement from ASTERISK-20158
There are several issues opened for bugs in app_alarmreceiver (ASTERISK-19435, ASTERISK-18417, ASTERISK-16694 and ASTERISK-16668), which go away after the cleanup (ASTERISK-16694) replacing the proprietary toneburst code with ast_playtones_ functions, so the fix from ASTERISK-19435 is no longer necessary.
The new functionality from ASTERISK-20158 is a bugfix on my opinion, because 'we get stuck on the alarm receiver app until it times out'


This addresses bugs ASTERISK-16668, ASTERISK-16694, ASTERISK-18417, ASTERISK-19435, ASTERISK-20157, ASTERISK-20158, and ASTERISK-20224.
    https://issues.asterisk.org/jira/browse/ASTERISK-16668
    https://issues.asterisk.org/jira/browse/ASTERISK-16694
    https://issues.asterisk.org/jira/browse/ASTERISK-18417
    https://issues.asterisk.org/jira/browse/ASTERISK-19435
    https://issues.asterisk.org/jira/browse/ASTERISK-20157
    https://issues.asterisk.org/jira/browse/ASTERISK-20158
    https://issues.asterisk.org/jira/browse/ASTERISK-20224


Diffs (updated)
-----

  /trunk/apps/app_alarmreceiver.c 370760 

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


Testing
-------

Compile test only


Thanks,

KNK

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120817/84f7e088/attachment.htm>


More information about the asterisk-dev mailing list