[asterisk-dev] [Code Review] SIP: more code refactoring, more unit tests

David Vossel dvossel at digium.com
Thu Feb 11 10:42:22 CST 2010


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

Review request for Asterisk Developers.


Summary
-------

----- Code Refactoring Changes ----
- read_to_parts() moved to reqresp_parser.c and has been renamed as get_name_and_number()
- get_in_brackets() moved to reqresp_parser.c
- find_closing_quotes() added to sip_utils.h

----- Logic Changes -----
- get_name_and_number() now uses parse_uri() and get_calleridname() for parsing.  Before this change only names within quotes were found, when names not within quotes are possible.

----- New Unit Tests -----
sip_get_name_and_number_test
sip_get_in_brackets_test


Diffs
-----

  /trunk/channels/chan_sip.c 246298 
  /trunk/channels/sip/include/reqresp_parser.h 246298 
  /trunk/channels/sip/include/sip_utils.h 246298 
  /trunk/channels/sip/reqresp_parser.c 246298 

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


Testing
-------

-tested sip redirect to verify get_name_and_number still works as expected.
-ran unit tests, verified they all passed.


Thanks,

David




More information about the asterisk-dev mailing list