new loplugin unnecessaryparen

Change-Id: Ic883a07b30069ca6342d7521c8ad890f4326f0ec
Reviewed-on: https://gerrit.libreoffice.org/39549
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

4 files changed, 16 insertions, 16 deletions

diff --git a/slideshow/source/engine/opengl/TransitionerImpl.cxx b/slideshow/source/engine/opengl/TransitionerImpl.cxx
index 68d50b9b03f3..95d97533325e 100644
--- a/slideshow/source/engine/opengl/TransitionerImpl.cxx
+++ b/slideshow/source/engine/opengl/TransitionerImpl.cxx
@@ -888,8 +888,8 @@ const OGLFormat* OGLTransitionerImpl::chooseFormats()
     uno::Reference<rendering::XIntegerBitmapColorSpace> xIntColorSpace(
         maSlideBitmapLayout.ColorSpace);
 
-    if( (xIntColorSpace->getType() == rendering::ColorSpaceType::RGB ||
-         xIntColorSpace->getType() == rendering::ColorSpaceType::SRGB) )
+    if( xIntColorSpace->getType() == rendering::ColorSpaceType::RGB ||
+        xIntColorSpace->getType() == rendering::ColorSpaceType::SRGB )
     {
         /* table for canvas->OGL format mapping. outer index is number
            of color components (0:3, 1:4), then comes bits per pixel
diff --git a/slideshow/source/engine/slide/slideimpl.cxx b/slideshow/source/engine/slide/slideimpl.cxx
index 95582c37b718..0f5515a0cae8 100644
--- a/slideshow/source/engine/slide/slideimpl.cxx
+++ b/slideshow/source/engine/slide/slideimpl.cxx
@@ -865,7 +865,7 @@ void SlideImpl::applyShapeAttributes(
             // not a shape target. Maybe a ParagraphTarget?
             presentation::ParagraphTarget aParaTarget;
 
-            if( (aProps[i].Target >>= aParaTarget) )
+            if( aProps[i].Target >>= aParaTarget )
             {
                 // yep, ParagraphTarget found - extract shape
                 // and index
diff --git a/slideshow/source/engine/slide/targetpropertiescreator.cxx b/slideshow/source/engine/slide/targetpropertiescreator.cxx
index 7bd686b42208..73635e0053f6 100644
--- a/slideshow/source/engine/slide/targetpropertiescreator.cxx
+++ b/slideshow/source/engine/slide/targetpropertiescreator.cxx
@@ -278,7 +278,7 @@ namespace internal
                             {
                                 // try to extract string
                                 OUString aString;
-                                if( (aAny >>= aString) )
+                                if( aAny >>= aString )
                                 {
                                     // we also take the strings "true" and "false",
                                     // as well as "on" and "off" here
diff --git a/slideshow/source/engine/tools.cxx b/slideshow/source/engine/tools.cxx
index 64d52465d2d7..3061dc485ef7 100644
--- a/slideshow/source/engine/tools.cxx
+++ b/slideshow/source/engine/tools.cxx
@@ -139,7 +139,7 @@ namespace slideshow
                            const ::basegfx::B2DVector&  rSlideBounds )
         {
             // try to extract numeric value (double, or smaller POD, like float or int)
-            if( (rSourceAny >>= o_rValue) )
+            if( rSourceAny >>= o_rValue)
             {
                 // succeeded
                 return true;
@@ -174,7 +174,7 @@ namespace slideshow
                            const ::basegfx::B2DVector&      /*rSlideBounds*/ )
         {
             // try to extract numeric value (int, or smaller POD, like byte)
-            if( (rSourceAny >>= o_rValue) )
+            if( rSourceAny >>= o_rValue)
             {
                 // succeeded
                 return true;
@@ -182,7 +182,7 @@ namespace slideshow
 
             // okay, no plain int. Maybe one of the domain-specific enums?
             drawing::FillStyle eFillStyle;
-            if( (rSourceAny >>= eFillStyle) )
+            if( rSourceAny >>= eFillStyle )
             {
                 o_rValue = sal::static_int_cast<sal_Int16>(eFillStyle);
 
@@ -191,7 +191,7 @@ namespace slideshow
             }
 
             drawing::LineStyle eLineStyle;
-            if( (rSourceAny >>= eLineStyle) )
+            if( rSourceAny >>= eLineStyle )
             {
                 o_rValue = sal::static_int_cast<sal_Int16>(eLineStyle);
 
@@ -200,7 +200,7 @@ namespace slideshow
             }
 
             awt::FontSlant eFontSlant;
-            if( (rSourceAny >>= eFontSlant) )
+            if( rSourceAny >>= eFontSlant )
             {
                 o_rValue = sal::static_int_cast<sal_Int16>(eFontSlant);
 
@@ -242,7 +242,7 @@ namespace slideshow
             // try to extract numeric value (double, or smaller POD, like float or int)
             {
                 double nTmp = 0;
-                if( (rSourceAny >>= nTmp) )
+                if( rSourceAny >>= nTmp )
                 {
                     sal_uInt32 aIntColor( static_cast< sal_uInt32 >(nTmp) );
 
@@ -257,7 +257,7 @@ namespace slideshow
             // try double sequence
             {
                 uno::Sequence< double > aTmp;
-                if( (rSourceAny >>= aTmp) )
+                if( rSourceAny >>= aTmp )
                 {
                     ENSURE_OR_THROW( aTmp.getLength() == 3,
                                       "extractValue(): inappropriate length for RGB color value" );
@@ -272,7 +272,7 @@ namespace slideshow
             // try sal_Int32 sequence
             {
                 uno::Sequence< sal_Int32 > aTmp;
-                if( (rSourceAny >>= aTmp) )
+                if( rSourceAny >>= aTmp )
                 {
                     ENSURE_OR_THROW( aTmp.getLength() == 3,
                                       "extractValue(): inappropriate length for RGB color value" );
@@ -292,7 +292,7 @@ namespace slideshow
             // try sal_Int8 sequence
             {
                 uno::Sequence< sal_Int8 > aTmp;
-                if( (rSourceAny >>= aTmp) )
+                if( rSourceAny >>= aTmp )
                 {
                     ENSURE_OR_THROW( aTmp.getLength() == 3,
                                       "extractValue(): inappropriate length for RGB color value" );
@@ -324,7 +324,7 @@ namespace slideshow
             // try double sequence
             {
                 uno::Sequence< double > aTmp;
-                if( (rSourceAny >>= aTmp) )
+                if( rSourceAny >>= aTmp )
                 {
                     ENSURE_OR_THROW( aTmp.getLength() == 3,
                                       "extractValue(): inappropriate length for HSL color value" );
@@ -339,7 +339,7 @@ namespace slideshow
             // try sal_Int8 sequence
             {
                 uno::Sequence< sal_Int8 > aTmp;
-                if( (rSourceAny >>= aTmp) )
+                if( rSourceAny >>= aTmp )
                 {
                     ENSURE_OR_THROW( aTmp.getLength() == 3,
                                       "extractValue(): inappropriate length for HSL color value" );
@@ -375,7 +375,7 @@ namespace slideshow
         {
             bool bTmp;
             // try to extract bool value
-            if( (rSourceAny >>= bTmp) )
+            if( rSourceAny >>= bTmp )
             {
                 o_rValue = bTmp;