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

Jonathan Rose reviewboard at asterisk.org
Fri Jun 20 15:46:37 CDT 2014


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

(Updated June 20, 2014, 3:46 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, kmoore and Matt Jordan.


Changes
-------

Committed in revision 416849


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/20140620/268c0085/attachment-0001.html>


More information about the asterisk-dev mailing list