<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hello,<br>
<br>
looking at the patch it seems that the new function does 'malloc'
and memset to zero of allocated space, am I right? If yes, this is
different that the standard calloc prototype/behaviour.<br>
<br>
It is better to rename it to something else, to let the option open
for adding an alternative to standard 'calloc'. I would suggest for
example pkg_mallocz() -- ending z to indicate that the memory chunk
is set to 0.<br>
<br>
Also, and alternative has to be added for shm manager. Your patch is
only for pkg, but we have to keep the two coherent. There are some
components where the memory functions are provided as parameter,
and, a matter of need, they can be the ones for pkg or shm.<br>
<br>
Cheers,<br>
Daniel<br>
<br>
<br>
<div class="moz-cite-prefix">On 12/21/12 10:32 PM, Konstantin M.
wrote:<br>
</div>
<blockquote
cite="mid:CAEYtTzfC0T6LCo=kLqeZ-rfWgGd52mUgFK+W7_nGqwspCe1o_g@mail.gmail.com"
type="cite">Hello,<br>
<br>
Here is proposed patch for adding calloc to pkg_ memory manager.<br>
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sr-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a>
<a class="moz-txt-link-freetext" href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a>
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Daniel-Constantin Mierla - <a class="moz-txt-link-freetext" href="http://www.asipto.com">http://www.asipto.com</a>
<a class="moz-txt-link-freetext" href="http://twitter.com/#!/miconda">http://twitter.com/#!/miconda</a> - <a class="moz-txt-link-freetext" href="http://www.linkedin.com/in/miconda">http://www.linkedin.com/in/miconda</a></pre>
</body>
</html>