diff options
Diffstat (limited to 'sc')
27 files changed, 83 insertions, 82 deletions
diff --git a/sc/qa/extras/check_data_pilot_table.cxx b/sc/qa/extras/check_data_pilot_table.cxx index f13768ffbc87..261fa23b303c 100644 --- a/sc/qa/extras/check_data_pilot_table.cxx +++ b/sc/qa/extras/check_data_pilot_table.cxx @@ -126,7 +126,7 @@ uno::Reference< uno::XInterface > CheckDataPilotTable::init() { oSheet->getCellByPosition(i, j)->setValue(i * (j + 1)); oSheet2->getCellByPosition(i, j)->setValue(i * (j + 2)); - } + } } // change a value of a cell and check the change in the data pilot diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx index 3191e3d5f816..1db11be0c777 100644 --- a/sc/source/core/data/documen2.cxx +++ b/sc/source/core/data/documen2.cxx @@ -835,7 +835,7 @@ bool ScDocument::CopyTab( SCTAB nOldPos, SCTAB nNewPos, const ScMarkData* pOnlyM if (pValidationList) pValidationList->UpdateInsertTab(aCxt); - } + } else bValid = false; } diff --git a/sc/source/core/data/postit.cxx b/sc/source/core/data/postit.cxx index 882cc8c190bb..a4ec79b2c909 100644 --- a/sc/source/core/data/postit.cxx +++ b/sc/source/core/data/postit.cxx @@ -462,7 +462,7 @@ ScCaptionPtr::ScCaptionPtr( SdrCaptionObj* p ) : if (p) { newHead(); - } + } } ScCaptionPtr::ScCaptionPtr( const ScCaptionPtr& r ) : @@ -805,7 +805,7 @@ void ScCaptionPtr::dissolve() ScCaptionPtr* p = pThat->mpNext; pThat->clear(); pThat = p; - } + } assert(!mpHead && !mpNext && !mpCaption); // should had been cleared during list walk delete pHead; } diff --git a/sc/source/core/data/table1.cxx b/sc/source/core/data/table1.cxx index 40aeab42cfa2..b0885b08e5cb 100644 --- a/sc/source/core/data/table1.cxx +++ b/sc/source/core/data/table1.cxx @@ -516,7 +516,7 @@ bool ScTable::GetCellArea( SCCOL& rEndCol, SCROW& rEndRow ) const SCCOL nMaxX = 0; SCROW nMaxY = 0; for (SCCOL i=0; i<aCol.size(); i++) - { + { if (!aCol[i].IsEmptyData()) { bFound = true; @@ -539,7 +539,7 @@ bool ScTable::GetCellArea( SCCOL& rEndCol, SCROW& rEndRow ) const nMaxX = i; } } - } + } rEndCol = nMaxX; rEndRow = nMaxY; diff --git a/sc/source/core/data/table2.cxx b/sc/source/core/data/table2.cxx index af36f60f0d9e..1074fe1f65a1 100644 --- a/sc/source/core/data/table2.cxx +++ b/sc/source/core/data/table2.cxx @@ -964,10 +964,10 @@ public: { mnFilteredRows++; continue; -} + } if (mbAsLink && mnFlags == InsertDeleteFlags::ALL) -{ + { // with InsertDeleteFlags::ALL, also create links (formulas) for empty cells setLink(nRow); continue; @@ -1043,12 +1043,12 @@ void ScTable::TransposeColPatterns(ScTable* pTransClip, SCCOL nCol1, SCCOL nCol, SCROW nRow2, SCROW nCombinedStartRow, bool bIncludeFiltered, const std::vector<SCROW>& rFilteredRows, SCROW nRowDestOffset) { - SCROW nAttrRow1 = {}; // spurious -Werror=maybe-uninitialized - SCROW nAttrRow2 = {}; // spurious -Werror=maybe-uninitialized - const ScPatternAttr* pPattern; - std::unique_ptr<ScAttrIterator> pAttrIter(aCol[nCol].CreateAttrIterator( nRow1, nRow2 )); - while ( (pPattern = pAttrIter->Next( nAttrRow1, nAttrRow2 )) != nullptr ) - { + SCROW nAttrRow1 = {}; // spurious -Werror=maybe-uninitialized + SCROW nAttrRow2 = {}; // spurious -Werror=maybe-uninitialized + const ScPatternAttr* pPattern; + std::unique_ptr<ScAttrIterator> pAttrIter(aCol[nCol].CreateAttrIterator( nRow1, nRow2 )); + while ( (pPattern = pAttrIter->Next( nAttrRow1, nAttrRow2 )) != nullptr ) + { if ( !IsDefaultItem( pPattern ) ) { const SfxItemSet& rSet = pPattern->GetItemSet(); diff --git a/sc/source/core/opencl/op_financial.cxx b/sc/source/core/opencl/op_financial.cxx index 6d44271f1203..30a47cbfbd93 100644 --- a/sc/source/core/opencl/op_financial.cxx +++ b/sc/source/core/opencl/op_financial.cxx @@ -412,11 +412,11 @@ void OpFV::GenSlidingWindowFunction(std::stringstream& ss, ss << " double arg4 = " << GetBottom() << ";\n"; unsigned j = vSubArguments.size(); while (j--) - { + { FormulaToken* pCur = vSubArguments[j]->GetFormulaToken(); assert(pCur); if(pCur->GetType() == formula::svSingleVectorRef) - { + { const formula::SingleVectorRefToken* pSVR = static_cast< const formula::SingleVectorRefToken* >(pCur); ss << " if(gid0 >= " << pSVR->GetArrayLength() << " || isnan("; @@ -427,8 +427,8 @@ void OpFV::GenSlidingWindowFunction(std::stringstream& ss, ss << " arg" << j << " = "; ss << vSubArguments[j]->GenSlidingWindowDeclRef(); ss << ";\n"; - } } + } ss << " tmp = GetFV(arg0, arg1, arg2, arg3, arg4);\n"; ss << " return tmp;\n"; ss << "}"; @@ -463,11 +463,11 @@ void OpIPMT::GenSlidingWindowFunction(std::stringstream& ss, ss << " double arg5 = " << GetBottom() << ";\n"; unsigned j = vSubArguments.size(); while (j--) - { + { FormulaToken* pCur = vSubArguments[j]->GetFormulaToken(); assert(pCur); if(pCur->GetType() == formula::svSingleVectorRef) - { + { const formula::SingleVectorRefToken* pSVR = static_cast< const formula::SingleVectorRefToken* >(pCur); ss << " if(gid0 >= " << pSVR->GetArrayLength() << " || isnan("; @@ -478,8 +478,8 @@ void OpIPMT::GenSlidingWindowFunction(std::stringstream& ss, ss << " arg" << j << " = "; ss << vSubArguments[j]->GenSlidingWindowDeclRef(); ss << ";\n"; - } } + } ss << " double pmt ;\n"; ss << " if(arg0 == 0.0)\n"; ss << " return 0;\n"; @@ -666,8 +666,8 @@ void OpDuration_ADD::GenSlidingWindowFunction(std::stringstream& ss, ss << " arg" << j << " = "; ss << vSubArguments[j]->GenSlidingWindowDeclRef(); ss << ";\n"; - } } + } ss << " int nNullDate = GetNullDate();\n"; ss << " tmp = GetDuration( nNullDate, (int)arg0, (int)arg1, arg2,"; ss << " arg3, (int)arg4, (int)arg5);\n"; @@ -4428,7 +4428,7 @@ tmpCur3); ss<< " int buffer_fv_len = "; ss<< tmpCurDVR3->GetArrayLength(); ss << ";\n"; - } + } if(vSubArguments.size()>4) { diff --git a/sc/source/core/opencl/op_logical.cxx b/sc/source/core/opencl/op_logical.cxx index 2d91b169723f..b9c70a1c7413 100644 --- a/sc/source/core/opencl/op_logical.cxx +++ b/sc/source/core/opencl/op_logical.cxx @@ -71,19 +71,19 @@ void OpAnd::GenSlidingWindowFunction(std::stringstream &ss, ss << "0; i < " << nCurWindowSize << "; i++) {\n"; } if(!pCurDVR->IsStartFixed() && !pCurDVR->IsEndFixed()) - { + { ss <<" if(isnan("<<vSubArguments[j]->GenSlidingWindowDeclRef(); ss <<")||i+gid0>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 1;\n else\n"; - } + } else - { + { ss <<" if(isnan("<<vSubArguments[j]->GenSlidingWindowDeclRef(); ss <<")||i>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 1;\n else\n"; - } + } ss <<" tmp = "; ss <<vSubArguments[j]->GenSlidingWindowDeclRef()<<";\n"; ss <<" tmp"<<j<<" = tmp"<<j<<" && tmp;\n"; @@ -156,19 +156,19 @@ void OpOr::GenSlidingWindowFunction(std::stringstream &ss, ss << "0; i < " << nCurWindowSize << "; i++) {\n"; } if(!pCurDVR->IsStartFixed() && !pCurDVR->IsEndFixed()) - { + { ss <<" if(isnan("<<vSubArguments[j]->GenSlidingWindowDeclRef(); ss <<")||i+gid0>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 0;\n else\n"; - } + } else - { + { ss <<" if(isnan("<<vSubArguments[j]->GenSlidingWindowDeclRef(); ss <<")||i>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 0;\n else\n"; - } + } ss <<" tmp = "; ss <<vSubArguments[j]->GenSlidingWindowDeclRef()<<";\n"; ss <<" tmp"<<j<<" = tmp"<<j<<" || tmp;\n"; @@ -272,19 +272,19 @@ void OpXor::GenSlidingWindowFunction(std::stringstream &ss, ss << "0; i < " << nCurWindowSize << "; i++) {\n"; } if(!pCurDVR->IsStartFixed() && !pCurDVR->IsEndFixed()) - { + { ss <<" if(isnan("<<rArg->GenSlidingWindowDeclRef(); ss <<")||i+gid0>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 0;\n else\n"; - } + } else - { + { ss <<" if(isnan("<<rArg->GenSlidingWindowDeclRef(); ss <<")||i>="<<pCurDVR->GetArrayLength(); ss <<")\n"; ss <<" tmp = 0;\n else\n"; - } + } ss <<" tmp = "; ss <<rArg->GenSlidingWindowDeclRef()<<";\n"; ss <<" tmp0 = (tmp != 0);\n"; diff --git a/sc/source/core/opencl/op_math.cxx b/sc/source/core/opencl/op_math.cxx index 01658ce987d1..4042e2511177 100644 --- a/sc/source/core/opencl/op_math.cxx +++ b/sc/source/core/opencl/op_math.cxx @@ -875,13 +875,13 @@ void OpAverageIfs::GenSlidingWindowFunction(std::stringstream &ss, tmpss << " tmp += tmp0;\n"; tmpss << " count++;\n"; for(size_t j=1;j<vSubArguments.size();j+=2,m--) - { + { for(int n = 0;n<m+1;n++) { tmpss << " "; } tmpss<< "}\n"; - } + } UnrollDoubleVector(ss,tmpss,pCurDVR,nCurWindowSize); diff --git a/sc/source/core/opencl/op_statistical.cxx b/sc/source/core/opencl/op_statistical.cxx index 273f37dcc204..3de4417fb240 100644 --- a/sc/source/core/opencl/op_statistical.cxx +++ b/sc/source/core/opencl/op_statistical.cxx @@ -3081,7 +3081,7 @@ void OpCorrel::GenSlidingWindowFunction( ss << "vYSum += pow(arg1 - vYMean, 2);\n\t\t\t"; ss << "vSum += (arg0 - vXMean)*(arg1 - vYMean);\n\t\t"; ss << "}\n\t\t"; - } else if (pCurDVRX->IsStartFixed() && !pCurDVRX->IsEndFixed()) { + } else if (pCurDVRX->IsStartFixed() && !pCurDVRX->IsEndFixed()) { ss << "0; i < gid0 + " << nCurWindowSizeX << "; i++) {\n\t\t\t"; ss << "arg0 = " << vSubArguments[0] ->GenSlidingWindowDeclRef(true) << ";\n\t\t\t"; @@ -4127,7 +4127,7 @@ void OpPermut::GenSlidingWindowFunction( if (i) ss << ","; vSubArguments[i]->GenSlidingWindowDecl(ss); - } + } ss << ") {\n"; ss <<" int gid0=get_global_id(0);\n"; ss <<" double inA;\n"; @@ -7353,7 +7353,7 @@ void OpDevSq::GenSlidingWindowFunction(std::stringstream& ss, if (i) ss << ","; vSubArguments[i]->GenSlidingWindowDecl(ss); - } + } ss << ") {\n"; ss << " int gid0 = get_global_id(0);\n"; ss << " double vSum = 0.0;\n"; @@ -7418,7 +7418,7 @@ void OpDevSq::GenSlidingWindowFunction(std::stringstream& ss, ss << " vSum += arg" << i << ";\n"; ss << " }\n"; } -} + } else if (pCur->GetType() == formula::svSingleVectorRef) { const formula::SingleVectorRefToken* pTVR = @@ -7449,7 +7449,7 @@ void OpDevSq::GenSlidingWindowFunction(std::stringstream& ss, ss << " cnt++;\n"; ss << " vSum += arg" << i << ";\n"; } - } + } ss << " vMean = vSum / cnt;\n"; ss << " vSum = 0.0;\n"; for(size_t k = 0; k < vSubArguments.size(); k++ ) @@ -7502,7 +7502,7 @@ void OpDevSq::GenSlidingWindowFunction(std::stringstream& ss, ss << " vSum += pow( arg" << k << " - vMean, 2 );\n"; ss << " }\n"; } - } + } else if (pCur->GetType() == formula::svSingleVectorRef) { const formula::SingleVectorRefToken* pTVR = @@ -7529,7 +7529,7 @@ void OpDevSq::GenSlidingWindowFunction(std::stringstream& ss, ss << vSubArguments[k]->GenSlidingWindowDeclRef() << ";\n"; ss << " vSum += pow( arg" << k << " - vMean, 2 );\n"; } - } + } ss << " return vSum;\n"; ss << "}"; } diff --git a/sc/source/filter/excel/excform8.cxx b/sc/source/filter/excel/excform8.cxx index 19f2a8c66257..62e184204a1c 100644 --- a/sc/source/filter/excel/excform8.cxx +++ b/sc/source/filter/excel/excform8.cxx @@ -1658,7 +1658,7 @@ void ExcelToSc8::GetAbsRefs( ScRangeList& r, XclImpStream& aIn, std::size_t nLen {// nFactor -> skip bytes or words AttrChoose nData++; nSeek = nData * 2; - } + } } break; } diff --git a/sc/source/filter/excel/excimp8.cxx b/sc/source/filter/excel/excimp8.cxx index e1f26807b556..d2c58f3d234d 100644 --- a/sc/source/filter/excel/excimp8.cxx +++ b/sc/source/filter/excel/excimp8.cxx @@ -360,7 +360,7 @@ void ImportExcel8::ReadBasic() aVbaPrj.setOleOverridesSink( xOleNameOverrideSink ); aVbaPrj.importVbaProject( *vbaStg ); GetObjectManager().SetOleNameOverrideInfo( xOleNameOverrideSink ); - } + } } catch( uno::Exception& ) { diff --git a/sc/source/filter/excel/xiescher.cxx b/sc/source/filter/excel/xiescher.cxx index 9b8d33f31d7d..0e8ff487e8b1 100644 --- a/sc/source/filter/excel/xiescher.cxx +++ b/sc/source/filter/excel/xiescher.cxx @@ -3686,7 +3686,7 @@ SdrObject* XclImpDffConverter::ProcessObj( SvStream& rDffStrm, DffObjData& rDffO but that should not have any effect for grouped objects. */ if( !bIsTopLevel ) xDrawObj->PostProcessSdrObject( *this, *xSdrObj ); - } + } return xSdrObj.release(); } diff --git a/sc/source/filter/excel/xlroot.cxx b/sc/source/filter/excel/xlroot.cxx index c232b6082b2f..a965a5e16d2c 100644 --- a/sc/source/filter/excel/xlroot.cxx +++ b/sc/source/filter/excel/xlroot.cxx @@ -388,7 +388,7 @@ ScHeaderEditEngine& XclRoot::GetHFEditEngine() const pEditSet->Put( aItemSet.Get( ATTR_CJK_FONT_HEIGHT ).CloneSetWhich(EE_CHAR_FONTHEIGHT_CJK) ); pEditSet->Put( aItemSet.Get( ATTR_CTL_FONT_HEIGHT ).CloneSetWhich(EE_CHAR_FONTHEIGHT_CTL) ); rEE.SetDefaults( std::move(pEditSet) ); // takes ownership - } + } return *mrData.mxHFEditEngine; } diff --git a/sc/source/ui/app/seltrans.cxx b/sc/source/ui/app/seltrans.cxx index 7c50c1cf1b6f..9b7b3caa0905 100644 --- a/sc/source/ui/app/seltrans.cxx +++ b/sc/source/ui/app/seltrans.cxx @@ -45,7 +45,7 @@ static bool lcl_IsURLButton( SdrObject* pObject ) SdrUnoObj* pUnoCtrl = dynamic_cast<SdrUnoObj*>( pObject ); if (pUnoCtrl && SdrInventor::FmForm == pUnoCtrl->GetObjInventor()) - { + { const uno::Reference<awt::XControlModel>& xControlModel = pUnoCtrl->GetUnoControlModel(); OSL_ENSURE( xControlModel.is(), "uno control without model" ); if ( xControlModel.is() ) @@ -61,7 +61,7 @@ static bool lcl_IsURLButton( SdrObject* pObject ) if ( (aAny >>= eTmp) && eTmp == form::FormButtonType_URL ) bRet = true; } - } + } } return bRet; diff --git a/sc/source/ui/miscdlgs/anyrefdg.cxx b/sc/source/ui/miscdlgs/anyrefdg.cxx index 6738eb79bd06..7e310d12cb45 100644 --- a/sc/source/ui/miscdlgs/anyrefdg.cxx +++ b/sc/source/ui/miscdlgs/anyrefdg.cxx @@ -133,7 +133,7 @@ void ScFormulaReferenceHelper::ShowSimpleReference(const OUString& rStr) ScRange const & rRangeEntry = aRangeList[ i ]; Color aColName = ScRangeFindList::GetColorName( i ); pTabViewShell->AddHighlightRange( rRangeEntry, aColName ); - } + } } } diff --git a/sc/source/ui/miscdlgs/dataproviderdlg.cxx b/sc/source/ui/miscdlgs/dataproviderdlg.cxx index 85d58a907527..d41c3842488c 100644 --- a/sc/source/ui/miscdlgs/dataproviderdlg.cxx +++ b/sc/source/ui/miscdlgs/dataproviderdlg.cxx @@ -677,7 +677,7 @@ std::shared_ptr<sc::DataTransformation> ScDateTimeTransformation::getTransformat // translate from 1-based column notations to internal Calc one aColumns.insert(nCol - 1); -} + } switch (nPos) { case 0: diff --git a/sc/source/ui/unoobj/chartuno.cxx b/sc/source/ui/unoobj/chartuno.cxx index 60e3d2aa4536..d336cdd201b8 100644 --- a/sc/source/ui/unoobj/chartuno.cxx +++ b/sc/source/ui/unoobj/chartuno.cxx @@ -482,7 +482,7 @@ void ScChartObj::GetData_Impl( ScRangeListRef& rRanges, bool& rColHeaders, bool& } bFound = true; } - } + } if( !bFound ) { rRanges = nullptr; diff --git a/sc/source/ui/unoobj/confuno.cxx b/sc/source/ui/unoobj/confuno.cxx index b95566ed667f..2f9550812260 100644 --- a/sc/source/ui/unoobj/confuno.cxx +++ b/sc/source/ui/unoobj/confuno.cxx @@ -584,7 +584,7 @@ uno::Any SAL_CALL ScDocumentConfiguration::getPropertyValue( const OUString& aPr break; } } - } + } } else diff --git a/sc/source/ui/unoobj/dapiuno.cxx b/sc/source/ui/unoobj/dapiuno.cxx index 6b2abcb752dc..9a85aae7cb8d 100644 --- a/sc/source/ui/unoobj/dapiuno.cxx +++ b/sc/source/ui/unoobj/dapiuno.cxx @@ -1670,7 +1670,8 @@ OUString SAL_CALL ScDataPilotFieldObj::getName() aName = *pLayoutName; else aName = pDim->GetName(); - } } + } + } return aName; } diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx index e12207f3ad37..5f625839f490 100644 --- a/sc/source/ui/unoobj/docuno.cxx +++ b/sc/source/ui/unoobj/docuno.cxx @@ -1182,11 +1182,11 @@ void ScModelObj::initializeForTiledRendering(const css::uno::Sequence<css::beans for (const beans::PropertyValue& rValue : rArguments) { if (rValue.Name == ".uno:SpellOnline" && rValue.Value.has<bool>()) - { + { ScDocOptions options = GetDocument()->GetDocOptions(); options.SetAutoSpell(rValue.Value.get<bool>()); GetDocument()->SetDocOptions(options); - } + } } // show us the text exactly diff --git a/sc/source/ui/vba/vbainterior.cxx b/sc/source/ui/vba/vbainterior.cxx index 241099a00624..9c99fc39e762 100644 --- a/sc/source/ui/vba/vbainterior.cxx +++ b/sc/source/ui/vba/vbainterior.cxx @@ -166,7 +166,7 @@ ScVbaInterior::GetColorIndex( const sal_Int32 nColor ) sal_Int32 nElems = xIndex->getCount(); sal_Int32 nIndex = -1; for ( sal_Int32 count=0; count<nElems; ++count ) - { + { sal_Int32 nPaletteColor = 0; xIndex->getByIndex( count ) >>= nPaletteColor; if ( nPaletteColor == nColor ) diff --git a/sc/source/ui/vba/vbarange.cxx b/sc/source/ui/vba/vbarange.cxx index 4837f5525aa3..aff1c4b00eff 100644 --- a/sc/source/ui/vba/vbarange.cxx +++ b/sc/source/ui/vba/vbarange.cxx @@ -200,24 +200,24 @@ static uno::Reference< excel::XRange > lcl_makeXRangeFromSheetCellRanges( const const uno::Sequence< table::CellRangeAddress > sAddresses = xLocSheetCellRanges->getRangeAddresses(); ScRangeList aCellRanges; if ( sAddresses.hasElements() ) - { - for ( const auto& rAddress : sAddresses ) { - ScRange refRange; - ScUnoConversion::FillScRange( refRange, rAddress ); - aCellRanges.push_back( refRange ); - } - // Single range - if ( aCellRanges.size() == 1 ) - { - uno::Reference< table::XCellRange > xTmpRange( new ScCellRangeObj( pDoc, aCellRanges.front() ) ); - xRange = new ScVbaRange( xParent, xContext, xTmpRange ); - } - else - { - uno::Reference< sheet::XSheetCellRangeContainer > xRanges( new ScCellRangesObj( pDoc, aCellRanges ) ); - xRange = new ScVbaRange( xParent, xContext, xRanges ); - } + for ( const auto& rAddress : sAddresses ) + { + ScRange refRange; + ScUnoConversion::FillScRange( refRange, rAddress ); + aCellRanges.push_back( refRange ); + } + // Single range + if ( aCellRanges.size() == 1 ) + { + uno::Reference< table::XCellRange > xTmpRange( new ScCellRangeObj( pDoc, aCellRanges.front() ) ); + xRange = new ScVbaRange( xParent, xContext, xTmpRange ); + } + else + { + uno::Reference< sheet::XSheetCellRangeContainer > xRanges( new ScCellRangesObj( pDoc, aCellRanges ) ); + xRange = new ScVbaRange( xParent, xContext, xRanges ); + } } return xRange; } diff --git a/sc/source/ui/view/dbfunc3.cxx b/sc/source/ui/view/dbfunc3.cxx index 3ca2bb5e7fe5..15141c68a064 100644 --- a/sc/source/ui/view/dbfunc3.cxx +++ b/sc/source/ui/view/dbfunc3.cxx @@ -1525,7 +1525,7 @@ void ScDBFunc::DataPilotInput( const ScAddress& rPos, const OUString& rString ) } else pErrorId = STR_INVALIDNAME; - } + } } else if (aPosData.Flags & MemberResultFlags::GRANDTOTAL) { diff --git a/sc/source/ui/view/editsh.cxx b/sc/source/ui/view/editsh.cxx index 98bf88d1f99f..f51e8f49290f 100644 --- a/sc/source/ui/view/editsh.cxx +++ b/sc/source/ui/view/editsh.cxx @@ -356,7 +356,7 @@ void ScEditShell::Execute( SfxRequest& rReq ) sInput.iterateCodePoints(&nUtf16Pos); if( nUtf16Pos > aSel.nEndPos ) aSel.nEndPos = nUtf16Pos; - } + } ToggleUnicodeCodepoint aToggle; while( nUtf16Pos && aToggle.AllowMoreInput( sInput[nUtf16Pos-1]) ) diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx index c2c115c4371c..ec90a6bac1bb 100644 --- a/sc/source/ui/view/gridwin.cxx +++ b/sc/source/ui/view/gridwin.cxx @@ -3382,7 +3382,7 @@ void ScGridWindow::KeyInput(const KeyEvent& rKEvt) SfxBindings& rBindings = pViewShell->GetViewFrame()->GetBindings(); rBindings.Invalidate(SID_ATTR_TRANSFORM_POS_X); rBindings.Invalidate(SID_ATTR_TRANSFORM_POS_Y); - } + } return; } diff --git a/sc/source/ui/view/preview.cxx b/sc/source/ui/view/preview.cxx index 0ac87b887c08..f2faf59edd6b 100644 --- a/sc/source/ui/view/preview.cxx +++ b/sc/source/ui/view/preview.cxx @@ -1230,7 +1230,7 @@ void ScPreview::MouseButtonUp( const MouseEvent& rMEvt ) bBottomRulerChange = false; bHeaderRulerChange = false; bFooterRulerChange = false; - } + } } bTopRulerMove = false; bBottomRulerMove = false; diff --git a/sc/source/ui/view/printfun.cxx b/sc/source/ui/view/printfun.cxx index dc084ec75689..9fb716331383 100644 --- a/sc/source/ui/view/printfun.cxx +++ b/sc/source/ui/view/printfun.cxx @@ -2544,10 +2544,10 @@ tools::Long ScPrintFunc::CountNotePages() if (rDoc.HasColNotes(nCol, nPrintTab)) { for ( SCROW nRow = nStartRow; nRow <= nEndRow; ++nRow ) - { - if ( rDoc.HasNote(nCol, nRow, nPrintTab) ) - aNotePosList.emplace_back( nCol, nRow, nPrintTab ); - } + { + if ( rDoc.HasNote(nCol, nRow, nPrintTab) ) + aNotePosList.emplace_back( nCol, nRow, nPrintTab ); + } } } } |