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>

6 files changed, 8 insertions, 8 deletions

diff --git a/sfx2/source/appl/appopen.cxx b/sfx2/source/appl/appopen.cxx
index 9821f16e56b0..647c7f7cec29 100644
--- a/sfx2/source/appl/appopen.cxx
+++ b/sfx2/source/appl/appopen.cxx
@@ -186,7 +186,7 @@ ErrCode CheckPasswd_Impl
 {
     ErrCode nRet = ERRCODE_NONE;
 
-    if( ( !pFile->GetFilter() || pFile->IsStorage() ) )
+    if( !pFile->GetFilter() || pFile->IsStorage() )
     {
         uno::Reference< embed::XStorage > xStorage = pFile->GetStorage();
         if( xStorage.is() )
diff --git a/sfx2/source/dialog/dockwin.cxx b/sfx2/source/dialog/dockwin.cxx
index a8c7143bbf18..445489490596 100644
--- a/sfx2/source/dialog/dockwin.cxx
+++ b/sfx2/source/dialog/dockwin.cxx
@@ -969,7 +969,7 @@ void SfxDockingWindow::Initialize(SfxChildWinInfo *pInfo)
     // detect floating mode
     // toggeling mode will not execute code in handlers, because pImpl->bConstructed is not set yet
     bool bFloatMode = IsFloatingMode();
-    if ( bFloatMode != ((GetAlignment() == SfxChildAlignment::NOALIGNMENT)) )
+    if ( bFloatMode != (GetAlignment() == SfxChildAlignment::NOALIGNMENT) )
     {
         bFloatMode = !bFloatMode;
         SetFloatingMode( bFloatMode );
diff --git a/sfx2/source/doc/guisaveas.cxx b/sfx2/source/doc/guisaveas.cxx
index bd3c541f8d81..77f4b8bcc2ae 100644
--- a/sfx2/source/doc/guisaveas.cxx
+++ b/sfx2/source/doc/guisaveas.cxx
@@ -224,7 +224,7 @@ public:
         catch( const uno::Exception& )
         {}
 
-        if ( ( bReadOnly && !m_bReadOnlySupported ) )
+        if ( bReadOnly && !m_bReadOnlySupported )
             throw uno::RuntimeException(); // the user could provide the data, so it must be stored
     }
 
diff --git a/sfx2/source/doc/iframe.cxx b/sfx2/source/doc/iframe.cxx
index 8070c90c1e2c..b6558f86672e 100644
--- a/sfx2/source/doc/iframe.cxx
+++ b/sfx2/source/doc/iframe.cxx
@@ -282,7 +282,7 @@ void SAL_CALL IFrameObject::setPropertyValue(const OUString& aPropertyName, cons
     case WID_FRAME_IS_AUTO_BORDER:
     {
         bool bIsAutoBorder;
-        if ( (aAny >>= bIsAutoBorder) )
+        if ( aAny >>= bIsAutoBorder )
         {
             bool bBorder = maFrmDescr.IsFrameBorderOn();
             maFrmDescr.ResetBorder();
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index 1bb7a932b853..5c3f71f1c398 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -800,13 +800,13 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed )
                 if ( xProps->hasPropertyByName( aAuthor ) )
                 {
                     aAny = aContent.getPropertyValue( aAuthor );
-                    if ( ( aAny >>= aValue ) )
+                    if ( aAny >>= aValue )
                         xDocProps->setAuthor(aValue);
                 }
                 if ( xProps->hasPropertyByName( aKeywords ) )
                 {
                     aAny = aContent.getPropertyValue( aKeywords );
-                    if ( ( aAny >>= aValue ) )
+                    if ( aAny >>= aValue )
                         xDocProps->setKeywords(
                           ::comphelper::string::convertCommaSeparated(aValue));
 ;
@@ -814,7 +814,7 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed )
                 if ( xProps->hasPropertyByName( aSubject ) )
                 {
                     aAny = aContent.getPropertyValue( aSubject );
-                    if ( ( aAny >>= aValue ) ) {
+                    if ( aAny >>= aValue ) {
                         xDocProps->setSubject(aValue);
                     }
                 }
diff --git a/sfx2/source/view/viewfrm2.cxx b/sfx2/source/view/viewfrm2.cxx
index b6e5b96a1e4d..2c63fdcae93d 100644
--- a/sfx2/source/view/viewfrm2.cxx
+++ b/sfx2/source/view/viewfrm2.cxx
@@ -233,7 +233,7 @@ void SfxViewFrame::Exec_Impl(SfxRequest &rReq )
 
                 // Document only needs to be queried, if no other View present.
                 bool bClosed = false;
-                if ( ( bOther || pDocSh->PrepareClose( true/*bUI*/ ) ) )
+                if ( bOther || pDocSh->PrepareClose( true/*bUI*/ ) )
                 {
                     if ( !bOther )
                         pDocSh->SetModified( false );