<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body style="word-wrap:break-word; color:rgb(0,0,0); font-size:14px; font-family:Calibri,sans-serif">
<div>This is commentary on sections 2.4 and 2.5 in reference to Bob Steagall’s review of the getdns API - <a href="http://www.vpnc.org/pipermail/getdns-api/2014-January/000229.html">http://www.vpnc.org/pipermail/getdns-api/2014-January/000229.html</a></div>
<br>
<div>Section 2.4 talks about the current convert functions.  I think it was a misunderstanding since the return value is actually allocated instead of modified in place.  That discussion should probably move to the Distinguish strings from wire format in conversion
 functions discussion as well.</div>
<div><br>
</div>
<div>Section 2.5 introduces a few additional functions:</div>
<div><br>
</div>
<div>
<div>char* getdns_convert_dns_name_to_fqdn_2 </div>
<div>( </div>
<div> char const* src_name, </div>
<div> size_t src_len, </div>
<div> char* dst_name, </div>
<div> size_t dst_len </div>
<div>); </div>
<div> </div>
<div>char* getdns_convert_fqdn_to_dns_name_2 </div>
<div>( </div>
<div> char const* src_name, </div>
<div> size_t src_len, </div>
<div> char* dst_name, </div>
<div> size_t dst_len </div>
<div>); </div>
<div> </div>
<div>char* getdns_convert_ulabel_to_alabel_2 </div>
<div>( </div>
<div> char const* src_ulabel, </div>
<div> size_t src_len, </div>
<div> char* dst_alabel, </div>
<div> size_t dst_len </div>
<div>); </div>
<div> </div>
<div>char* getdns_convert_alabel_to_ulabel_2 </div>
<div>( </div>
<div> char const* src_alabel, </div>
<div> size_t src_len, </div>
<div> char* dst_ulabel, </div>
<div> size_t dst_len </div>
<div>); </div>
</div>
<div><br>
</div>
<div>
<div>Note that the size of the source array is provided in the examples above. While one could make an </div>
<div>argument against including this extra parameter, it does have the advantages of allowing for early </div>
<div>parameter validation and permitting the use of embedded substrings as source arguments. </div>
</div>
<div><br>
</div>
<div>I am not sure these need to be included initially.  The big benefit to me is the ability to convert substrings - is that something we think the users might need?  If we were to include these, then I think we should tweak the signatures to distinguish between
 wire format and string and also return getdns_return_t.</div>
<div><br>
</div>
<h5><font color="gray">“This message (including any attachments) is intended only for the use of the individual or entity to which it is addressed, and may contain information that is non-public, proprietary, privileged, confidential and exempt from disclosure
 under applicable law or may be constituted as attorney work product. If you are not the intended recipient, you are hereby notified that any use, dissemination, distribution, or copying of this communication is strictly prohibited. If you have received this
 message in error, notify sender immediately and delete this message immediately.”
</h5>
</font>
</body>
</html>