[asterisk-dev] Re: russell: trunk r65968 - in /trunk: apps/ channels/ include/asterisk/ main/ pbx/

[Russell Bryant]

Tony Mountifield wrote:
> Scanning through the patch, this was the only instance I saw that passed
> a pointer to an attr as the second arg instead of NULL.
> 
> Is there any reason the attr is part of the conf structure? It just appears
> to get inited and destroyed in the same way as a temporary one would, and
> every other call to ast_pthread_create_background() was content to let it
> use a local temporary attr.

The argument is there in case someone wants to use the API call but set more 
thread attributes than just the detached attribute.  It's true that there aren't 
many places in the code that use it today, but it just seemed like the right 
thing to do.

-- 
Russell Bryant
Software Engineer
Digium, Inc.