[Libreoffice-commits] core.git: cppu/source

Stephan Bergmann sbergman at redhat.com
Tue Mar 8 17:16:22 UTC 2016


 cppu/source/uno/data.cxx |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit c2fe73d6a976e5a9131ff4aa04e7b5cea674ef19
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Mar 8 18:15:52 2016 +0100

    Remove redundant checks (that trigger loplugin:implicitboolconversion])
    
    Change-Id: I86c0b56b7d330e3e3450a5806a89ca8fb7a1bfae

diff --git a/cppu/source/uno/data.cxx b/cppu/source/uno/data.cxx
index 088f176..9a33a2a 100644
--- a/cppu/source/uno/data.cxx
+++ b/cppu/source/uno/data.cxx
@@ -468,10 +468,10 @@ public:
 };
 BinaryCompatible_Impl::BinaryCompatible_Impl()
 {
-    static_assert( ((sal_Bool) true) == sal_True &&
-                         (1 != 0) == sal_True, "must be binary compatible" );
-    static_assert( ((sal_Bool) false) == sal_False &&
-                         (1 == 0) == sal_False, "must be binary compatible" );
+    static_assert( static_cast<sal_Bool>(true) == sal_True,
+                   "must be binary compatible" );
+    static_assert( static_cast<sal_Bool>(false) == sal_False,
+                   "must be binary compatible" );
 #ifdef MAX_ALIGNMENT_4
     // max alignment is 4
     static_assert(offsetof(AlignSize_Impl, dDouble) == 4, "offsetof(AlignSize_Impl, dDouble) != 4");


More information about the Libreoffice-commits mailing list