[1350] Changes after review:

- len() returns now uint16_t type
- using pre-increment intead of post-increment
- using distance() function
- moved some simple implementation to header

1 files changed, 2 insertions, 6 deletions

diff --git a/src/lib/dhcp/option4_addrlst.h b/src/lib/dhcp/option4_addrlst.h
index a6fbb0f9db..c7958056f0 100644
--- a/src/lib/dhcp/option4_addrlst.h
+++ b/src/lib/dhcp/option4_addrlst.h
@@ -53,7 +53,7 @@ public:
     /// @param addrs container with a list of addresses
     Option4AddrLst(uint8_t type, const AddressContainer& addrs);
 
-    /// @brief Constrcutor, creates an option with a single address.
+    /// @brief Constructor, creates an option with a single address.
     ///
     /// Creates an option that contains a single address.
     ///
@@ -63,9 +63,6 @@ public:
 
     /// @brief Constructor, used for received options.
     ///
-    /// This contructor is similar to the previous one, but it does not take
-    /// the whole vector<uint8_t>, but rather subset of it.
-    ///
     /// TODO: This can be templated to use different containers, not just
     /// vector. Prototype should look like this:
     /// template<typename InputIterator> Option(Universe u, uint16_t type,
@@ -106,8 +103,7 @@ public:
     /// option header)
     ///
     /// @return length of the option
-    virtual unsigned short
-    len();
+    virtual uint16_t len();
 
     /// @brief Returns vector with addresses.
     ///