daemon: dangerous operator bool()
Issue generated from Tuleap's migration script. Originally submitted by: Guillaume Roguez (guillaume)
operator bool() SHALL always be marked as "explicit" to prevent unattended behavior, like calling == operators. In such case, lhs and rhs are casted as bool (implicit cast) before the ==, almost resulting on a unattended behavior. As an boggus behavior, see UPnPContext::removeMapping() doing a if (mapping == \*global\_mapping) at line 431.
arguments type inherits of Mapping class.
This one doesn't explicit its overloaded bool operator causing the previous if to always be true (as both arguments are valid).