Re: [asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-19 Thread opticron
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3079/#review10461 --- Ship it! Ship It! - opticron On Dec. 18, 2013, 6:21 p.m.,

Re: [asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-19 Thread Matt Jordan
On Dec. 19, 2013, 9:10 a.m., opticron wrote: Ship It! Well, hold on. Don't Ship It! just yet. :-) The fact that this isn't called in Asterisk 1.8 doesn't mean it shouldn't be called - it just means someone, at some point in time, fixed the glitch. If we aren't calling this function, then

Re: [asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-19 Thread Joshua Colp
On Dec. 19, 2013, 3:10 p.m., opticron wrote: Ship It! Matt Jordan wrote: Well, hold on. Don't Ship It! just yet. :-) The fact that this isn't called in Asterisk 1.8 doesn't mean it shouldn't be called - it just means someone, at some point in time, fixed the glitch. If

Re: [asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-19 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3079/ --- (Updated Dec. 19, 2013, 5:26 p.m.) Status -- This change has been

[asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-18 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3079/ --- Review request for Asterisk Developers. Repository: Asterisk

Re: [asterisk-dev] [Code Review] 3079: udptl: Dead code elimination. ast_udptl_bridge not used.

2013-12-18 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3079/ --- (Updated Dec. 19, 2013, 12:21 a.m.) Review request for Asterisk