[asterisk-dev] [Code Review] Add to Stasis subscription change notices and message cache dumping

opticron reviewboard at asterisk.org
Wed Mar 6 11:45:32 CST 2013


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

(Updated March 6, 2013, 11:45 a.m.)


Review request for Asterisk Developers and David Lee.


Changes
-------

Update to diff against newly updated stasis-core.  Subscription change notices are now incorporated into stasis_core.


Summary (updated)
-------

This review covers several items involved in the transition of MWI from the event system to Stasis:
* Creation of cache dump capability (necessary for pollmailboxes in app_voicemail)
* Update of the Stasis unit tests to cover those new items
* Ability to set a server EID on messages


This addresses bug ASTERISK-21097.
    https://issues.asterisk.org/jira/browse/ASTERISK-21097


Diffs (updated)
-----

  team/dlee/stasis-core/include/asterisk/stasis.h 382509 
  team/dlee/stasis-core/main/stasis_cache.c 382509 
  team/dlee/stasis-core/main/stasis_message.c 382509 
  team/dlee/stasis-core/tests/test_stasis.c 382509 

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


Testing
-------

Tested this using the unit tests and in conjunction with the MWI changes they were created for.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130306/1c3002d6/attachment.htm>


More information about the asterisk-dev mailing list