Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-07 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/ --- (Updated March 7, 2014, 9:07 p.m.) Status -- This change has been

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Joshua Colp
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/#review11094 --- /branches/12/main/sorcery.c

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/ --- (Updated March 6, 2014, 5:27 p.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/#review11103 --- /branches/12/main/sorcery.c

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/ --- (Updated March 6, 2014, 8:17 p.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/#review6 --- Ship it! Minor issue below. /branches/12/main/sorcery.c

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Matt Jordan
On March 6, 2014, 3:03 p.m., rmudgett wrote: /branches/12/main/sorcery.c, line 311 https://reviewboard.asterisk.org/r/3305/diff/3/?file=55576#file55576line311 This function is using spaces and not tabs. Copying directly from the wiki pages always give spaces even though I

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Mark Michelson
On March 6, 2014, 9:03 p.m., rmudgett wrote: /branches/12/main/sorcery.c, line 311 https://reviewboard.asterisk.org/r/3305/diff/3/?file=55576#file55576line311 This function is using spaces and not tabs. Copying directly from the wiki pages always give spaces even though I

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-06 Thread Mark Michelson
On March 6, 2014, 9:03 p.m., rmudgett wrote: /branches/12/main/sorcery.c, line 311 https://reviewboard.asterisk.org/r/3305/diff/3/?file=55576#file55576line311 This function is using spaces and not tabs. Copying directly from the wiki pages always give spaces even though I

[asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-05 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/ --- Review request for Asterisk Developers and Joshua Colp. Repository:

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-05 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/#review11078 --- Another thing I neglected to mention in my review that

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-05 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3305/#review11080 --- /branches/12/main/sorcery.c

Re: [asterisk-dev] [Code Review] 3305: Sorcery: Be lenient when extra realtime fields are retrieved.

2014-03-05 Thread rmudgett
On March 5, 2014, 5:50 p.m., Mark Michelson wrote: Another thing I neglected to mention in my review that concerns me is the performance hit this patch could cause. Currently, the object fields on a sorcery object type are an ao2_container that is a linked list. The algorithm for