From: Marcin Siodelski <marcin@isc.org>
Date: Wed, 28 Nov 2012 09:57:23 +0000 (+0100)
Subject: [2365] Added todo comments for added functions.
X-Git-Tag: bind10-1.0.0-beta-release~46^2~19^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=31b4c8a77f74ff4f417171af1ceb428c134572a3;p=thirdparty%2Fkea.git

[2365] Added todo comments for added functions.
---

diff --git a/src/lib/dhcp/option.h b/src/lib/dhcp/option.h
index 963f93542b..a6b062286e 100644
--- 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.