This is silly and makes it harder to read:

return _M_value != 0 ? true : false;
Tested powerpc64le-linux, committed to trunk.


commit ca8d1e0e92810ac72b337247c10af1de1de465d4
Author: Jonathan Wakely <jwak...@redhat.com>
Date:   Thu Sep 17 15:07:24 2015 +0100

    Remove redundant conditional expressions in <system_error>
    
    	* include/std/system_error (error_code::operator bool(),
    	error_condition::operator bool()): Remove redundant conditional
    	expression.

diff --git a/libstdc++-v3/include/std/system_error b/libstdc++-v3/include/std/system_error
index 92f8af9..cc82bdf 100644
--- a/libstdc++-v3/include/std/system_error
+++ b/libstdc++-v3/include/std/system_error
@@ -181,7 +181,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
     { return category().message(value()); }
 
     explicit operator bool() const noexcept
-    { return _M_value != 0 ? true : false; }
+    { return _M_value != 0; }
 
     // DR 804.
   private:
@@ -257,7 +257,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
     { return category().message(value()); }
 
     explicit operator bool() const noexcept
-    { return _M_value != 0 ? true : false; }
+    { return _M_value != 0; }
 
     // DR 804.
   private:

Reply via email to