Re: [asterisk-dev] [Code Review] 4159: chan_sip: Add missing braces on if, else, while and for

2014-11-11 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4159/ --- (Updated Nov. 11, 2014, 2:17 p.m.) Status -- This change has been dis

Re: [asterisk-dev] [Code Review] 4159: chan_sip: Add missing braces on if, else, while and for

2014-11-11 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4159/#review13723 --- I fall into the camp of being not in favor of this patch. My ma

[asterisk-dev] [Code Review] 4159: chan_sip: Add missing braces on if, else, while and for

2014-11-09 Thread Corey Farrell
--- 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://i