Amebis/MacStd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

common: Require explicit handle validation

Browse Source 
Using operator bool() hid ambiguity when handle was polymorfic with
bool. Using operator!() reqired !! to test for validity which results in
awkward code.

Signed-off-by: Simon Rozman <simon@rozman.si>
This commit is contained in:
Simon Rozman 2025-06-09 22:40:25 +02:00
parent a04c27a14e
commit 8ed062347e
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
include/MacStd/common.hpp
View File

@ -173,18 +173,17 @@ namespace macstd
} }
/// ///
/// Tests if the object handle is invalid. /// Tests if the object handle is valid.
/// ///
/// \return /// \return
/// - Non zero when object handle is invalid; /// - Non zero when object handle is valid;
/// - Zero otherwise. /// - Zero otherwise.
/// ///
/// \note Implementing `operator bool() const` would be correct C++ approach here. Unfortunately, it would produce ambiguities where `bool` and `T` are polymorphed. /// \note Implementing `operator bool() const` would be correct C++ approach here. Unfortunately, it would produce ambiguities where `bool` and `T` are polymorphed.
/// Use `!!` to test if the object handle is valid.
/// ///
bool operator!() const bool valid() const
{ {
return m_h == INVAL; return m_h != INVAL;
} }
/// ///