[asterisk-dev] [Code Review] 3626: res_parking: Parking manager actions were registered without module information.

[Jonathan Rose]

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

Review request for Asterisk Developers, kmoore and Matt Jordan.


Repository: Asterisk


Description
-------

The manager actions for res_parking were being registered with ast_manager_register_xml_core instead of ast_manager_register_xml. Fixing this ran into a small snag when I realized that I couldn't just switch to ast_manager_register_xml since it isn't the same file that actually creates the module, so I'm getting the module info using the same methodology as what was used for registering the core parking functions (include/asterisk/parking.h) and using ast_manager_register2 instead.


Diffs
-----

  /branches/12/res/parking/parking_manager.c 416556 

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


Testing
-------

Made sure Asterisk doesn't crash on load, made sure the manager commands are registered and have the appropriate documentation.


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140617/30789045/attachment.html>