Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-27 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/ --- (Updated March 27, 2015, 2:06 a.m.) Status -- This change has been

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-27 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/#review14881 --- Before committing I found a link error for utils/refcounter

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-25 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/#review14835 --- Ship it! Minor nit. /branches/13/main/utils.c

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
On March 24, 2015, 5:08 p.m., rmudgett wrote: /branches/13/main/logger.c, lines 1331-1332 https://reviewboard.asterisk.org/r/4502/diff/2/?file=72787#file72787line1331 I don't think this thread needs to be treated differently than any other thread. The recursion check is thread

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/ --- (Updated March 24, 2015, 6:05 p.m.) Review request for Asterisk

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
On March 24, 2015, 1:52 p.m., rmudgett wrote: /branches/13/main/logger.c, line 1762 https://reviewboard.asterisk.org/r/4502/diff/1/?file=72474#file72474line1762 Why couldn't the log safe recursion check be done all the time by ast_log_full()? Relying on users to know

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread rmudgett
On March 24, 2015, 12:52 p.m., rmudgett wrote: /branches/13/main/logger.c, line 1762 https://reviewboard.asterisk.org/r/4502/diff/1/?file=72474#file72474line1762 Why couldn't the log safe recursion check be done all the time by ast_log_full()? Relying on users to

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
On March 24, 2015, 1:52 p.m., rmudgett wrote: /branches/13/main/logger.c, line 1762 https://reviewboard.asterisk.org/r/4502/diff/1/?file=72474#file72474line1762 Why couldn't the log safe recursion check be done all the time by ast_log_full()? Relying on users to know

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
On March 24, 2015, 4:13 p.m., rmudgett wrote: /branches/13/include/asterisk/utils.h, line 28 https://reviewboard.asterisk.org/r/4502/diff/1/?file=72471#file72471line28 This can be removed as it was added for the backtrace _ast_mem_backtrace_buffer[]. Ooops, it actually needs to

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/#review14808 --- /branches/13/include/asterisk/utils.h

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/ --- (Updated March 24, 2015, 4:31 p.m.) Review request for Asterisk

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4502/#review14802 --- /branches/13/main/logger.c

Re: [asterisk-dev] [Code Review] 4502: Improved and portable ast_log recursion avoidance

2015-03-24 Thread rmudgett
On March 24, 2015, 12:52 p.m., rmudgett wrote: /branches/13/main/logger.c, line 1762 https://reviewboard.asterisk.org/r/4502/diff/1/?file=72474#file72474line1762 Why couldn't the log safe recursion check be done all the time by ast_log_full()? Relying on users to