[asterisk-dev] [Code Review] 3127: Protect CEL data structures during reload and shutdown. v11 version.

2014-01-14 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3127/ --- Review request for Asterisk Developers. Bugs: AST-1253 https://issues.

Re: [asterisk-dev] [Code Review] 3127: Protect CEL data structures during reload and shutdown. v11 version.

2014-01-23 Thread opticron
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3127/#review10664 --- /branches/11/main/cel.c

Re: [asterisk-dev] [Code Review] 3127: Protect CEL data structures during reload and shutdown. v11 version.

2014-01-23 Thread rmudgett
> On Jan. 23, 2014, 11:22 a.m., opticron wrote: > > /branches/11/main/cel.c, line 228 > > > > > > The locking and unlocking here seems excessive. Is there a reason to > > maintain the lock beyond iterator creation?

Re: [asterisk-dev] [Code Review] 3127: Protect CEL data structures during reload and shutdown. v11 version.

2014-01-23 Thread opticron
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3127/#review10666 --- Ship it! Ship It! - opticron On Jan. 14, 2014, 4:24 p.m., r

Re: [asterisk-dev] [Code Review] 3127: Protect CEL data structures during reload and shutdown. v11 version.

2014-01-24 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3127/ --- (Updated Jan. 24, 2014, 5:18 p.m.) Status -- This change has been mar