[2365] Added todo comments for added functions.

diff --git a/src/lib/dhcp/option.h b/src/lib/dhcp/option.h
index 963f93542b0b9fc92a52626fce061339f0edca40..a6b062286e987e42117d657cb4b5ee2b61a26bf9 100644 (file)
--- a/src/lib/dhcp/option.h
+++ b/src/lib/dhcp/option.h
@@ -320,6 +320,11 @@ protected:
     /// derived classes that override pack.
     ///
     /// @param [out] buf output buffer.
+    ///
+    /// @todo The set of exceptions thrown by this function depend on
+    /// exceptions thrown by pack methods invoked on objects
+    /// representing sub options. We should consider whether to aggregate
+    /// those into one exception which can be documented here.
     void packOptions(isc::util::OutputBuffer& buf);
 
     /// @brief Builds a collection of sub options from the buffer.
@@ -329,6 +334,11 @@ protected:
     /// different exceptions when option assembly fails.
     ///
     /// @param buf buffer to be parsed.
+    ///
+    /// @todo The set of exceptions thrown by this function depend on
+    /// exceptions thrown by unpack methods invoked on objects
+    /// representing sub options. We should consider whether to aggregate
+    /// those into one exception which can be documented here.
     void unpackOptions(const OptionBuffer& buf);
 
     /// @brief A private method used for option correctness.