[3089] Added ticket number to commentary

Added todo to dns_client.cc commentary regarding
trac# 3286 and the behavior isc::dns::Message::fromWire.

diff --git a/src/bin/d2/dns_client.cc b/src/bin/d2/dns_client.cc
index 00b0ff989c165dbdacacf4c94963735bd49ecfe3..9202e7d2339990523d92fec4b6dca837373b78d3 100644 (file)
--- a/src/bin/d2/dns_client.cc
+++ b/src/bin/d2/dns_client.cc
@@ -50,8 +50,10 @@ public:
     // once in the lifetime of a Message instance.  Therefore, response_ is a
     // pointer reference thus allowing this class to replace the object
     // pointed to with a new Message instance each time a message is
-    // received. This allows a single DNSClientImpl instance to be used in
-    // for multiple, sequential IOFetch calls.
+    // received. This allows a single DNSClientImpl instance to be used for
+    // multiple, sequential IOFetch calls. (@todo Trac# 3286 has been opened
+    // against dns::Message::fromWire.  Should the behavior of fromWire change
+    // the behavior here with could be rexamined).
     D2UpdateMessagePtr& response_;
     // A caller-supplied external callback which is invoked when DNS message
     // exchange is complete or interrupted.

diff --git a/src/bin/d2/tests/nc_test_utils.h b/src/bin/d2/tests/nc_test_utils.h
index 6292436d675118148f78625606e7f65f865a196a..a99752528cd9ad5848c86469d43f5b14bba93531 100644 (file)
--- a/src/bin/d2/tests/nc_test_utils.h
+++ b/src/bin/d2/tests/nc_test_utils.h
@@ -358,6 +358,10 @@ extern void addDomainServer(DdnsDomainPtr& domain, const std::string& name,
 
 /// @brief Creates a hex text dump of the given data buffer.
 ///
+/// This method is not used for testing but is handy for debugging.  It creates
+/// a pleasantly formatted string of 2-digits per byte separated by spaces with
+/// 16 bytes per line.
+///
 /// @param data pointer to the data to dump
 /// @param len size (in bytes) of data
 extern std::string toHexText(const uint8_t* data, size_t len);