[asterisk-dev] [Code Review] 3104: PJSIP CLI Part 2

George Joseph reviewboard at asterisk.org
Thu Jan 16 12:04:34 CST 2014


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

(Updated Jan. 16, 2014, 11:04 a.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed comments.


Repository: Asterisk


Description
-------

Adds identify, transport and registration support to the CLI.
Creates 3 additional callbacks, one for an iterator, one for a comparator and one for a container.  This eliminates the link dependency from higher level modules to lower level ones.
Eliminates duplicate sorting.
Cleans up output formatting.
Pushes cli registration down to the implementing source file.
Adds several ast_sip_destroy_sorcery functions to complement existing ast_sip_sorcery_initialize functions.  The destroy functions unregister cli commands and cli formatters.


Diffs (updated)
-----

  branches/12/res/res_pjsip_outbound_registration.c 405643 
  branches/12/res/res_pjsip_endpoint_identifier_ip.c 405643 
  branches/12/res/res_pjsip/pjsip_options.c 405643 
  branches/12/res/res_pjsip/pjsip_configuration.c 405643 
  branches/12/res/res_pjsip/pjsip_cli.c 405643 
  branches/12/res/res_pjsip/location.c 405643 
  branches/12/res/res_pjsip/include/res_pjsip_private.h 405643 
  branches/12/res/res_pjsip/config_transport.c 405643 
  branches/12/res/res_pjsip/config_global.c 405643 
  branches/12/res/res_pjsip/config_domain_aliases.c 405643 
  branches/12/res/res_pjsip/config_auth.c 405643 
  branches/12/include/asterisk/res_pjsip_cli.h 405643 
  branches/12/include/asterisk/res_pjsip.h 405643 

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


Testing
-------


Thanks,

George Joseph

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


More information about the asterisk-dev mailing list