[#3991] Commented CWE 476 nullPointer

diff --git a/src/lib/util/optional.h b/src/lib/util/optional.h
index 16d5dc022adde3dc6c6cdf78d4a3dbaf2db4ba3f..40c6df221d0bcffc0910a92434294ecd8cb19221 100644 (file)
--- a/src/lib/util/optional.h
+++ b/src/lib/util/optional.h
@@ -91,6 +91,10 @@ public:
     /// with an appropriate constructor, the caller must create a
     /// template specialization similar to the one provided for
     /// @c std::string below.
+    ///
+    /// @todo: when C++17 will be required can be solved using "if contexpr"
+    /// for type convertible from int or default constructible.
+    /// This will fix complains from code checkers e.g. cppcheck CWE 476.
     Optional()
         : default_(T(0)), unspecified_(true) {
     }