diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-20 10:26:01 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-11-20 12:31:32 +0100 |
commit | 4f4486c61d3e437059a2ac77aae012489f5c7e25 (patch) | |
tree | 408c9a71e3a2d2c4a921ce3a376546ccadae4437 /sfx2 | |
parent | fda2ee3d87600ce7ecbec5528ea80b8e9758f584 (diff) |
look for =() in loplugin:unnecessaryparen
Change-Id: I4f9b71ff7767e90987bb40358fc46ed5d1d571d0
Reviewed-on: https://gerrit.libreoffice.org/44944
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sfx2')
-rw-r--r-- | sfx2/source/doc/DocumentMetadataAccess.cxx | 8 | ||||
-rw-r--r-- | sfx2/source/doc/printhelper.cxx | 8 | ||||
-rw-r--r-- | sfx2/source/view/sfxbasecontroller.cxx | 2 | ||||
-rw-r--r-- | sfx2/source/view/viewprn.cxx | 4 |
4 files changed, 11 insertions, 11 deletions
diff --git a/sfx2/source/doc/DocumentMetadataAccess.cxx b/sfx2/source/doc/DocumentMetadataAccess.cxx index 50e6572b57d0..efb3496820aa 100644 --- a/sfx2/source/doc/DocumentMetadataAccess.cxx +++ b/sfx2/source/doc/DocumentMetadataAccess.cxx @@ -246,8 +246,8 @@ splitPath(OUString const & i_rPath, // input must not start or end with '/' return false; } else { - o_rDir = (i_rPath.copy(0, idx)); - o_rRest = (i_rPath.copy(idx+1)); + o_rDir = i_rPath.copy(0, idx); + o_rRest = i_rPath.copy(idx+1); return true; } } @@ -260,8 +260,8 @@ splitXmlId(OUString const & i_XmlId, if ((idx <= 0) || (idx >= i_XmlId.getLength() - 1)) { return false; } else { - o_StreamName = (i_XmlId.copy(0, idx)); - o_Idref = (i_XmlId.copy(idx+1)); + o_StreamName = i_XmlId.copy(0, idx); + o_Idref = i_XmlId.copy(idx+1); return isValidXmlId(o_StreamName, o_Idref); } } diff --git a/sfx2/source/doc/printhelper.cxx b/sfx2/source/doc/printhelper.cxx index 15b87f304ca1..296c53da0aa7 100644 --- a/sfx2/source/doc/printhelper.cxx +++ b/sfx2/source/doc/printhelper.cxx @@ -277,16 +277,16 @@ uno::Sequence< beans::PropertyValue > SAL_CALL SfxPrintHelper::getPrinter() uno::Sequence< beans::PropertyValue > aPrinter(8); aPrinter.getArray()[7].Name = "CanSetPaperSize"; - aPrinter.getArray()[7].Value <<= ( pPrinter->HasSupport( PrinterSupport::SetPaperSize ) ); + aPrinter.getArray()[7].Value <<= pPrinter->HasSupport( PrinterSupport::SetPaperSize ); aPrinter.getArray()[6].Name = "CanSetPaperFormat"; - aPrinter.getArray()[6].Value <<= ( pPrinter->HasSupport( PrinterSupport::SetPaper ) ); + aPrinter.getArray()[6].Value <<= pPrinter->HasSupport( PrinterSupport::SetPaper ); aPrinter.getArray()[5].Name = "CanSetPaperOrientation"; - aPrinter.getArray()[5].Value <<= ( pPrinter->HasSupport( PrinterSupport::SetOrientation ) ); + aPrinter.getArray()[5].Value <<= pPrinter->HasSupport( PrinterSupport::SetOrientation ); aPrinter.getArray()[4].Name = "IsBusy"; - aPrinter.getArray()[4].Value <<= ( pPrinter->IsPrinting() ); + aPrinter.getArray()[4].Value <<= pPrinter->IsPrinting(); aPrinter.getArray()[3].Name = "PaperSize"; awt::Size aSize = impl_Size_Object2Struct(pPrinter->GetPaperSize() ); diff --git a/sfx2/source/view/sfxbasecontroller.cxx b/sfx2/source/view/sfxbasecontroller.cxx index 72ca5708d872..145caa12f666 100644 --- a/sfx2/source/view/sfxbasecontroller.cxx +++ b/sfx2/source/view/sfxbasecontroller.cxx @@ -710,7 +710,7 @@ Reference< frame::XDispatch > SAL_CALL SfxBaseController::queryDispatch( const SfxChildWindow* pChildWin = pFrame->GetChildWindow( SID_BROWSER ); Reference < frame::XFrame > xFrame; if ( pChildWin ) - xFrame = ( pChildWin->GetFrame() ); + xFrame = pChildWin->GetFrame(); if ( xFrame.is() ) xFrame->setName( sTargetFrameName ); diff --git a/sfx2/source/view/viewprn.cxx b/sfx2/source/view/viewprn.cxx index 904a5ea9fc38..2456829d52bb 100644 --- a/sfx2/source/view/viewprn.cxx +++ b/sfx2/source/view/viewprn.cxx @@ -704,14 +704,14 @@ void SfxViewShell::ExecPrint_Impl( SfxRequest &rReq ) aProps[nProp]. Name = "Wait"; bool bAsynchron = false; aProps[nProp].Value >>= bAsynchron; - aProps[nProp].Value <<= (!bAsynchron); + aProps[nProp].Value <<= !bAsynchron; } else if ( aProps[nProp].Name == "Silent" ) { aProps[nProp]. Name = "MonitorVisible"; bool bPrintSilent = false; aProps[nProp].Value >>= bPrintSilent; - aProps[nProp].Value <<= (!bPrintSilent); + aProps[nProp].Value <<= !bPrintSilent; } } } |