[Asterisk-Dev] any reason for #define FREE in the code ?

Kevin P. Fleming kpfleming at digium.com
Wed Dec 28 16:04:03 MST 2005


Luigi Rizzo wrote:

> ok but then i'd rather define the wrapper globally e.g. ast_free()
> and let those function call the wrapper.
> Even with the current solution you still don't get to see
> the __FILE__ and __LINE__ of the "calling" function,
> just those of the wrapper, which doesn't help too much.

I would agree with the latter part of your statement, but I don't see 
much value in the first part. There is no reason to force every other 
module to have to use ast_free() (or document when it is necessary and 
when it isn't).

With so few calls to FREE, my personal opinion is that this doesn't even 
warrant the time we've already spent discussing it :-)



More information about the asterisk-dev mailing list