Changeset 21369 in vbox for trunk/include

Timestamp:

Jul 7, 2009 6:52:12 PM (16 years ago)

Author:

vboxsync

Message:

IPRT: hide ministring::operator bool() to force people to use isEmpty() explicitly, second try

Location:

trunk/include

Files:

• VBox/com/string.h (modified) (1 diff)
• iprt/ministring_cpp.h (modified) (1 diff)

trunk/include/VBox/com/string.h

@@ -596 +596 @@
 // work around error C2593 of the stupid MSVC 7.x ambiguity resolver
 WORKAROUND_MSVC7_ERROR_C2593_FOR_BOOL_OP (Bstr)
-WORKAROUND_MSVC7_ERROR_C2593_FOR_BOOL_OP (Utf8Str)
 
 ////////////////////////////////////////////////////////////////////////////////

trunk/include/iprt/ministring_cpp.h

@@ -308 +308 @@
     bool operator>(const char *that) const { return compare(that) > 0; }
 
-    operator bool() const { return !isNull(); }
-
 protected:
+
+    /**
+     * Hide operator bool() to force people to use isEmpty() explicitly.
+     */
+    operator bool() const { return false; }
+
     /**
      *  Destructor implementation, also used to clean up in operator=()