Re: [asterisk-dev] [Code Review] 3984: Module to add devstate for MWI.

2014-09-12 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3984/#review13293 --- Ship it! Aside from what Matt had to say, this looks good by m

Re: [asterisk-dev] [Code Review] 3984: Module to add devstate for MWI.

2014-09-09 Thread Matt Jordan
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3984/#review13267 --- This looks really good. Nice job! It'd be good to get a test f

Re: [asterisk-dev] [Code Review] 3984: Module to add devstate for MWI.

2014-09-08 Thread Jason Parker
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3984/ --- (Updated Sept. 8, 2014, 1:21 p.m.) Review request for Asterisk Developers.

[asterisk-dev] [Code Review] 3984: Module to add devstate for MWI.

2014-09-08 Thread Jason Parker
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3984/ --- Review request for Asterisk Developers. Repository: Asterisk Description