Remove TODO comments about using explicit bool operators

CHANGES

@@ -1,4 +1,8 @@
 
+2.5-392 | 2018-01-19 11:39:34 -0600
+
+  * Make nearly all bool conversion operators explicit. (Corelight)
+
 2.5-390 | 2018-01-17 16:09:55 -0600
 
   * Logging: implement get_filter_names and small fixes.

VERSION

@@ -1 +1 @@
-2.5-390
+2.5-392

src/UID.h

@@ -59,7 +59,6 @@ public:
 	/**
 	 * @return false if the UID instance was created via the default ctor
 	 *         and not yet initialized w/ Set().
-	 * TODO: this would be better as an "explicit" conversion operator (C++11)
 	 */
 	explicit operator bool() const
 		{ return initialized; }

src/analyzer/Tag.h

@@ -42,9 +42,6 @@ public:
 	/**
 	 * Returns false if the tag represents an error value rather than a
 	 * legal analyzer type.
-	 * TODO: make this conversion operator "explicit" (C++11) or use a
-	 *       "safe bool" idiom (not necessary if "explicit" is available),
-	 *       otherwise this may allow nonsense/undesired comparison operations.
 	 */
 	explicit operator bool() const	{ return *this != Tag(); }
 

src/file_analysis/Tag.h

@@ -41,10 +41,6 @@ public:
 	/**
 	 * Returns false if the tag represents an error value rather than a
 	 * legal analyzer type.
-	 * TODO: make this conversion operator "explicit" (C++11) or use a
-	 *       "safe bool" idiom (not necessary if "explicit" is available),
-	 *       otherwise this may allow nonsense/undesired comparison operations.
-	 *
 	 */
 	explicit operator bool() const	{ return *this != Tag(); }