From 4b2262ab5b10f334f1984fec84d2978db81c58f1 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Wed, 5 Jul 2017 08:32:57 +0200 Subject: new loplugin unnecessaryparen Change-Id: Ic883a07b30069ca6342d7521c8ad890f4326f0ec Reviewed-on: https://gerrit.libreoffice.org/39549 Tested-by: Jenkins Reviewed-by: Noel Grandin --- .../source/engine/opengl/TransitionerImpl.cxx | 4 ++-- slideshow/source/engine/slide/slideimpl.cxx | 2 +- .../engine/slide/targetpropertiescreator.cxx | 2 +- slideshow/source/engine/tools.cxx | 24 +++++++++++----------- 4 files changed, 16 insertions(+), 16 deletions(-) (limited to 'slideshow') 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 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(eFillStyle); @@ -191,7 +191,7 @@ namespace slideshow } drawing::LineStyle eLineStyle; - if( (rSourceAny >>= eLineStyle) ) + if( rSourceAny >>= eLineStyle ) { o_rValue = sal::static_int_cast(eLineStyle); @@ -200,7 +200,7 @@ namespace slideshow } awt::FontSlant eFontSlant; - if( (rSourceAny >>= eFontSlant) ) + if( rSourceAny >>= eFontSlant ) { o_rValue = sal::static_int_cast(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; -- cgit