-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4159/
-----------------------------------------------------------
Review request for Asterisk Developers.
Bugs: ASTERISK-24510
https://issues.asterisk.org/jira/browse/ASTERISK-24510
Repository: Asterisk
Description
-------
Thank you in advance to anyone who takes the time to review this tedious patch.
It is large but benign.
Scanned for / fixed instances of the regex search pattern
(else|(if|while|for)\s*\(.*\))(\s*/\*.*\*/|)\s*$ in channels/chan_sip.c and
channels/sip/*.c.
I've proposed this change for 13 and trunk. The set of braces for one 'if'
statement was excluded from this change to allow the entire patch to apply
cleanly to 13 and trunk with patch fuzz level 0. If you approve this change
please note if you are approving it for 13+, or just trunk. I was late to
AstriDevCon so I'm not sure if this is acceptable to 13, figured it doesn't
hurt to ask. Asterisk 13 will be supported for a long time and chan_sip is in
extended support status. I feel that adding the braces will make it
easier/lower risk for me to fix bugs that are found in the next 4 years.
I have no idea what reviewboard is doing with the display for lines 6078 -
6254, no code is actually moved. Check the diff file for the actual changes in
that section.
Diffs
-----
/branches/13/channels/sip/reqresp_parser.c 427641
/branches/13/channels/sip/dialplan_functions.c 427641
/branches/13/channels/sip/config_parser.c 427641
/branches/13/channels/chan_sip.c 427641
Diff: https://reviewboard.asterisk.org/r/4159/diff/
Testing
-------
Compiled, visually inspected. Ran all of tests/channel/SIP in testsuite.
Thanks,
Corey Farrell
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
