[getdns-api] A couple of more changes
Matt Miller
linuxwolf at outer-planes.net
Wed Nov 20 09:06:55 MST 2013
On Nov 20, 2013, at 8:54 AM, Willem Toorop <willem at nlnetlabs.nl> wrote:
> op 20-11-13 16:30, Paul Hoffman schreef:
>> On Nov 19, 2013, at 8:45 PM, Melinda Shore <melinda.shore at nomountain.net> wrote:
>>> 2) From the sync functions we'd like to remove the the
>>> response_length parameter and pass the response parameter
>>> by reference (**response rather than *response)
>>
>> If you do #2, you need to add a new error that say "I couldn't allocate enough memory to give you a response", and you should document that you will not give partial responses in that case. (Please don't say "if **response is NULL, that tells you that it failed"; that's too 90's for this spec.)
>
> +1
As long as the memory allocation and deallocation stay at the same API layer (if it gets allocated by getdns, then it gets deallocated by getdns), this makes a lot of sense to me.
- m&m
Matthew A. Miller
< http://goo.gl/LK55L >
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 496 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://www.vpnc.org/pipermail/getdns-api/attachments/20131120/3f5422f9/attachment.bin>
More information about the getdns-api
mailing list