[asterisk-dev] Re: [asterisk-commits] russell: branch 1.2
r38288 - /branches/1.2/channels/chan_sip.c
Russell Bryant
russell at digium.com
Wed Jul 26 23:00:56 MST 2006
On Wed, 2006-07-26 at 21:01 -0500, Kevin P. Fleming wrote:
> Should we just move this into a common place, now that it's being repeated in quite a few places?
I certainly think that would be a good idea. I just couldn't decide
where to put it. Then, I just decided to at least have the bugs fixed
until I had an idea for what to do with it.
> Alternatively, since we have ast_malloc/ast_calloc, should we have ast_free (not in 1.2, of course)?
I suppose. However, I'm not sure how beneficial it would be other than
for matching the naming convention of the allocation wrappers. There is
obviously no error checking we can do on a call to free(), so we can't
inject an Asterisk log message or anything.
--
Russell Bryant
Software Developer
Digium, Inc.
More information about the asterisk-dev
mailing list