add comments

lib/dns/include/dns/rdata.h

@@ -291,7 +291,6 @@ dns_result_t dns_rdata_fromtext(dns_rdata_t *rdata,
  */
 
 dns_result_t dns_rdata_totext(dns_rdata_t *rdata,
-			      dns_name_t *origin,
 			      isc_buffer_t *target);
 /*
  * Convert 'rdata' into text format, storing the result in 'target'.
@@ -300,6 +299,9 @@ dns_result_t dns_rdata_totext(dns_rdata_t *rdata,
  *	If 'origin' is not NULL, then any names in the rdata that are
  *	subdomains of 'origin' will be made relative it.
  *
+ *	XXX Do we *really* want to support 'origin'?  I'm inclined towards "no"
+ *	at the moment.
+ *
  * Requires:
  *
  *	'rdata' is a valid, non-empty rdata

lib/dns/include/dns/rdataset.h

@@ -191,6 +191,8 @@ dns_rdataset_totext(dns_rdataset_t *rdataset,
  * Notes:
  *	The rdata cursor position will be changed.
  *
+ *	XXX may need to add 'origin' parameter if we go with that in rdata.
+ *
  * Requires:
  *	'rdataset' is a valid rdataset.
  *