[getdns-api] A couple of more changes

[Paul Hoffman]

On Nov 19, 2013, at 8:45 PM, Melinda Shore <melinda.shore at nomountain.net> wrote:

> 1) We'd like to remove getdns_free_sync_request_memory(),
>   as getdns_dict_destroy() can be used to release the memory
>   in question, and
> 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.)

--Paul Hoffman
_______________________________________________
getdns-api mailing list
getdns-api at vpnc.org