dfaure added a comment.
Urgh you don't need a new variable for failure(), just do bool failed() const { return !m_success && !m_canceled; } REVISION DETAIL https://phabricator.kde.org/D6829 To: chinmoyr, dfaure, #frameworks Cc: #frameworks
dfaure added a comment.
Urgh you don't need a new variable for failure(), just do bool failed() const { return !m_success && !m_canceled; } REVISION DETAIL https://phabricator.kde.org/D6829 To: chinmoyr, dfaure, #frameworks Cc: #frameworks