diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-05 13:44:17 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-06 06:47:06 +0100 |
| commit | a2e3705d8b3b05ae664d54b762d6ff72927d5e48 (patch) | |
| tree | 6ac31f9c9b419b12dbf065845a6941c143f57e2c /sc | |
| parent | f423ac2d3bdba4263f1f41e31e7e2b7715afdd6e (diff) | |
loplugin:unnecessaryparen improve member expression
Change-Id: I304621018cb1e2a47e478e86df4229bcf2176741
Reviewed-on: https://gerrit.libreoffice.org/68757
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/core/opencl/formulagroupcl.cxx | 2 | ||||
| -rw-r--r-- | sc/source/ui/dbgui/tpsubt.cxx | 4 | ||||
| -rw-r--r-- | sc/source/ui/vba/vbarange.cxx | 2 | ||||
| -rw-r--r-- | sc/source/ui/view/output3.cxx | 8 |
4 files changed, 8 insertions, 8 deletions
diff --git a/sc/source/core/opencl/formulagroupcl.cxx b/sc/source/core/opencl/formulagroupcl.cxx index 51e0ea4ec813..d9d7da3d501c 100644 --- a/sc/source/core/opencl/formulagroupcl.cxx +++ b/sc/source/core/opencl/formulagroupcl.cxx @@ -1712,7 +1712,7 @@ public: } // set kernel arg SAL_INFO("sc.opencl", "Kernel " << k << " arg " << argno << ": cl_mem: " << mpClmem2); - err = clSetKernelArg(k, argno, sizeof(cl_mem), &(mpClmem2)); + err = clSetKernelArg(k, argno, sizeof(cl_mem), &mpClmem2); if (CL_SUCCESS != err) throw OpenCLError("clSetKernelArg", err, __FILE__, __LINE__); return 1; diff --git a/sc/source/ui/dbgui/tpsubt.cxx b/sc/source/ui/dbgui/tpsubt.cxx index f4bbec1a7351..94f6f95cbb83 100644 --- a/sc/source/ui/dbgui/tpsubt.cxx +++ b/sc/source/ui/dbgui/tpsubt.cxx @@ -100,7 +100,7 @@ void ScTpSubTotalGroup::Init() GetItemSet().Get( nWhichSubTotals )); pViewData = rSubTotalItem.GetViewData(); - pDoc = ( pViewData ) ? pViewData->GetDocument() : nullptr; + pDoc = pViewData ? pViewData->GetDocument() : nullptr; OSL_ENSURE( pViewData && pDoc, "ViewData or Document not found :-(" ); @@ -473,7 +473,7 @@ void ScTpSubTotalOptions::Init() GetItemSet().Get( nWhichSubTotals )); pViewData = rSubTotalItem.GetViewData(); - pDoc = ( pViewData ) ? pViewData->GetDocument() : nullptr; + pDoc = pViewData ? pViewData->GetDocument() : nullptr; OSL_ENSURE( pViewData && pDoc, "ViewData or Document not found!" ); diff --git a/sc/source/ui/vba/vbarange.cxx b/sc/source/ui/vba/vbarange.cxx index fba198b5ba55..f1500352d4fd 100644 --- a/sc/source/ui/vba/vbarange.cxx +++ b/sc/source/ui/vba/vbarange.cxx @@ -678,7 +678,7 @@ public: { if ( !hasMoreElements() ) throw container::NoSuchElementException(); - CellPos aPos = *(m_it)++; + CellPos aPos = *m_it++; uno::Reference< table::XCellRange > xRangeArea = getArea( aPos.m_nArea ); uno::Reference< table::XCellRange > xCellRange( xRangeArea->getCellByPosition( aPos.m_nCol, aPos.m_nRow ), uno::UNO_QUERY_THROW ); diff --git a/sc/source/ui/view/output3.cxx b/sc/source/ui/view/output3.cxx index 270d974e3a0b..17f315ca14f9 100644 --- a/sc/source/ui/view/output3.cxx +++ b/sc/source/ui/view/output3.cxx @@ -77,7 +77,7 @@ Point ScOutputData::PrePrintDrawingLayer(long nLogStX, long nLogStY ) if(pViewShell || pDrawView) { - SdrView* pLocalDrawView = (pDrawView) ? pDrawView : pViewShell->GetSdrView(); + SdrView* pLocalDrawView = pDrawView ? pDrawView : pViewShell->GetSdrView(); if(pLocalDrawView) { @@ -115,7 +115,7 @@ void ScOutputData::PostPrintDrawingLayer(const Point& rMMOffset) // #i74768# if(pViewShell || pDrawView) { - SdrView* pLocalDrawView = (pDrawView) ? pDrawView : pViewShell->GetSdrView(); + SdrView* pLocalDrawView = pDrawView ? pDrawView : pViewShell->GetSdrView(); if(pLocalDrawView) { @@ -139,7 +139,7 @@ void ScOutputData::PrintDrawingLayer(SdrLayerID nLayer, const Point& rMMOffset) if(pViewShell || pDrawView) { - SdrView* pLocalDrawView = (pDrawView) ? pDrawView : pViewShell->GetSdrView(); + SdrView* pLocalDrawView = pDrawView ? pDrawView : pViewShell->GetSdrView(); if(pLocalDrawView) { @@ -197,7 +197,7 @@ void ScOutputData::DrawSelectiveObjects(SdrLayerID nLayer) if(pViewShell || pDrawView) { - SdrView* pLocalDrawView = (pDrawView) ? pDrawView : pViewShell->GetSdrView(); + SdrView* pLocalDrawView = pDrawView ? pDrawView : pViewShell->GetSdrView(); if(pLocalDrawView) { |