This is an automated email from the ASF dual-hosted git repository. mseidel pushed a commit to branch AOO42X in repository https://gitbox.apache.org/repos/asf/openoffice.git
The following commit(s) were added to refs/heads/AOO42X by this push: new 5f05c48 Fixed typos, small cleanup 5f05c48 is described below commit 5f05c48025b9d52fe720b8fd15975d5a4a9fd712 Author: mseidel <msei...@apache.org> AuthorDate: Tue Aug 24 14:18:49 2021 +0200 Fixed typos, small cleanup (cherry picked from commit 6e8337c105d646591dfdf02e780094cc04f2ad6a) --- main/sal/inc/osl/thread.hxx | 11 +-- main/sw/source/core/layout/frmtool.cxx | 141 +++++++++++++++++---------------- 2 files changed, 77 insertions(+), 75 deletions(-) diff --git a/main/sal/inc/osl/thread.hxx b/main/sal/inc/osl/thread.hxx index 30911df..5a98b63 100644 --- a/main/sal/inc/osl/thread.hxx +++ b/main/sal/inc/osl/thread.hxx @@ -67,11 +67,11 @@ public: sal_Bool SAL_CALL create() { OSL_ASSERT(m_hThread == 0); // only one running thread per instance - if (m_hThread) + if( m_hThread ) return sal_False; m_hThread = osl_createSuspendedThread( threadFunc, (void*)this); - if ( m_hThread ) + if( m_hThread ) osl_resumeThread(m_hThread); return m_hThread != 0; @@ -80,10 +80,11 @@ public: sal_Bool SAL_CALL createSuspended() { OSL_ASSERT(m_hThread == 0); // only one running thread per instance - if( m_hThread) + if( m_hThread ) return sal_False; - m_hThread= osl_createSuspendedThread( threadFunc, - (void*)this); + + m_hThread= osl_createSuspendedThread( threadFunc, (void*)this); + return m_hThread != 0; } diff --git a/main/sw/source/core/layout/frmtool.cxx b/main/sw/source/core/layout/frmtool.cxx index ff7c550..799e4c6 100644 --- a/main/sw/source/core/layout/frmtool.cxx +++ b/main/sw/source/core/layout/frmtool.cxx @@ -1,5 +1,5 @@ /************************************************************** - * + * * Licensed to the Apache Software Foundation (ASF) under one * or more contributor license agreements. See the NOTICE file * distributed with this work for additional information @@ -7,18 +7,20 @@ * to you under the Apache License, Version 2.0 (the * "License"); you may not use this file except in compliance * with the License. You may obtain a copy of the License at - * + * * http://www.apache.org/licenses/LICENSE-2.0 - * + * * Unless required by applicable law or agreed to in writing, * software distributed under the License is distributed on an * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY * KIND, either express or implied. See the License for the * specific language governing permissions and limitations * under the License. - * + * *************************************************************/ + + // MARKER(update_precomp.py): autogen include statement, do not remove #include "precompiled_sw.hxx" @@ -104,7 +106,7 @@ SwFrmNotify::SwFrmNotify( SwFrm *pF ) : aPrt( pF->Prt() ), bInvaKeep( sal_False ), bValidSize( pF->GetValidSizeFlag() ), - mbFrmDeleted( false ) // #i49383# + mbFrmDeleted( false ) // #i49383# { if ( pF->IsTxtFrm() ) { @@ -576,7 +578,7 @@ SwLayNotify::~SwLayNotify() pLay->InvaPercentLowers( pLay->Prt().Height() - aPrt.Height() ); } if ( pLay->IsTabFrm() ) - //Damit _nur_ der Shatten bei Groessenaenderungen gemalt wird. + //Damit _nur_ der Schatten bei Groessenaenderungen gemalt wird. ((SwTabFrm*)pLay)->SetComplete(); else { @@ -616,7 +618,7 @@ SwLayNotify::~SwLayNotify() if ( !IsLowersComplete() && !(pLay->GetType()&(FRM_FLY|FRM_SECTION) && pLay->Lower() && pLay->Lower()->IsColumnFrm()) && - (bPos || bNotify) && !(pLay->GetType() & 0x1823) ) //Tab, Row, FtnCont, Root, Page + (bPos || bNotify) && !(pLay->GetType() & 0x1823) ) //Tab, Row, FtnCont, Root, Page { // --> OD 2005-03-11 #i44016# - force unlock of position of lower objects. // --> OD 2005-03-30 #i43913# - no unlock of position of objects, @@ -835,7 +837,7 @@ SwCntntNotify::~SwCntntNotify() pCell = pCell->GetUpper(); ASSERT( pCell->IsCellFrm(), "Where's my cell?" ); if ( text::VertOrientation::NONE != pCell->GetFmt()->GetVertOrient().GetVertOrient() ) - pCell->InvalidatePrt(); //fuer vertikale Ausrichtung. + pCell->InvalidatePrt(); // for vertical alignment } // OD 2004-02-26 #i25029# @@ -891,7 +893,7 @@ SwCntntNotify::~SwCntntNotify() { //Aktive PlugIn's oder OLE-Objekte sollten etwas von der Veraenderung //mitbekommen, damit sie Ihr Window entsprechend verschieben. - ViewShell *pSh = pCnt->getRootFrm()->GetCurrShell(); + ViewShell *pSh = pCnt->getRootFrm()->GetCurrShell(); if ( pSh ) { SwOLENode *pNd; @@ -900,8 +902,8 @@ SwCntntNotify::~SwCntntNotify() pNd->IsOLESizeInvalid()) ) { // --> OD #i117189# - const bool bNoTxtFrmPrtAreaChanged = - ( aPrt.SSize().Width() != 0 && + const bool bNoTxtFrmPrtAreaChanged = + ( aPrt.SSize().Width() != 0 && aPrt.SSize().Height() != 0 ) && aPrt.SSize() != pCnt->Prt().SSize(); // <-- @@ -919,7 +921,7 @@ SwCntntNotify::~SwCntntNotify() // if the frame is formatted for the first time. // Unfortunately this is not valid anymore since the // SwNoTxtFrm already gets a width during CalcLowerPreps. - // Nevertheless, the indention of !bFirst seemed to be + // Nevertheless, the intention of !bFirst seemed to be // to assure that the OLE objects have already been notified // if necessary before calling CalcAndSetScale. // So I replaced !bFirst by !IsOLESizeInvalid. There is @@ -931,7 +933,7 @@ SwCntntNotify::~SwCntntNotify() // uiview/view.cxx. if ( !pNd->IsOLESizeInvalid() && !pSh->GetDoc()->IsUpdateExpFld() ) - pFESh->CalcAndSetScale( xObj, + pFESh->CalcAndSetScale( xObj, &pFly->Prt(), &pFly->Frm(), bNoTxtFrmPrtAreaChanged ); } @@ -963,16 +965,16 @@ SwCntntNotify::~SwCntntNotify() if ( pDoc->GetSpzFrmFmts()->Count() && pDoc->DoesContainAtPageObjWithContentAnchor() && !pDoc->IsNewDoc() ) { - // If certain import filters for foreign file format import - // AT_PAGE anchored objects, the corresponding page number is + // If certain import filters for foreign file format import + // AT_PAGE anchored objects, the corresponding page number is // typically not known. In this case the content position is - // stored at which the anchored object is found in the + // stored at which the anchored object is found in the // imported document. - // When this content is formatted it is the time at which + // When this content is formatted it is the time at which // the page is known. Thus, this data can be corrected now. const SwPageFrm* pPage = 0; - SwNodeIndex *pIdx = 0; + SwNodeIndex *pIdx = 0; SwSpzFrmFmts *pTbl = pDoc->GetSpzFrmFmts(); for ( sal_uInt16 i = 0; i < pTbl->Count(); ++i ) { @@ -1179,9 +1181,9 @@ void AppendAllObjs( const SwSpzFrmFmts *pTbl, const SwFrm* pSib ) { //Verbinden aller Objekte, die in der SpzTbl beschrieben sind mit dem //Layout. - //Wenn sich nix mehr tut hoeren wir auf. Dann koennen noch Formate - //uebrigbleiben, weil wir weder zeichengebunde Rahmen verbinden noch - //Objecte die in zeichengebundenen verankert sind. + //Wenn sich nichts mehr tut hoeren wir auf. Dann koennen noch Formate + //uebrigbleiben, weil wir weder zeichengebundene Rahmen verbinden noch + //Objekte die in zeichengebundenen verankert sind. SwSpzFrmFmts aCpy( 255, 255 ); aCpy.Insert( pTbl, 0 ); @@ -1263,9 +1265,9 @@ void MA_FASTCALL _InsertCnt( SwLayoutFrm *pLay, SwDoc *pDoc, //lange Kette von Absaetzen teuer verschoben werden muss, bis sie sich auf //ertraegliches mass reduziert hat. //Wir gehen mal davon aus, da? 20 Absaetze auf eine Seite passen - //Damit es in extremen Faellen nicht gar so heftig rechenen wir je nach + //Damit es in extremen Faellen nicht gar so heftig rechnen wir je nach //Node noch etwas drauf. - //Wenn in der DocStatistik eine brauchebare Seitenzahl angegeben ist + //Wenn in der DocStatistik eine brauchbare Seitenzahl angegeben ist //(wird beim Schreiben gepflegt), so wird von dieser Seitenanzahl //ausgegengen. const sal_Bool bStartPercent = bPages && !nEndIndex; @@ -1996,7 +1998,7 @@ long SwBorderAttrs::CalcLeft( const SwFrm *pCaller ) const |* |* Beschreibung Berechnung der Groessen fuer Umrandung und Schatten. |* Es kann auch ohne Linien ein Abstand erwuenscht sein, -|* dieser wird dann nicht vom Attribut sondern hier +|* dieser wird dann nicht vom Attribut sondern hier |* beruecksichtigt (bBorderDist, z.B. fuer Zellen). |* |*************************************************************************/ @@ -2058,7 +2060,7 @@ void SwBorderAttrs::_IsLine() |* Zusaetzlich muss der Absatz mindestens rechts oder links oder |* unten eine Umrandung haben. |* 2. Die Umrandung unten faellt weg, wenn der Nachfolger dieselbe -|* Umrandung untern aufweist und 3. Zustrifft. +|* Umrandung unten aufweist und 3. zutrifft. |* Zusaetzlich muss der Absatz mindestens rechts oder links oder |* oben eine Umrandung haben. |* 3. Die Umrandungen links und rechts vor Vorgaenger bzw. Nachfolger @@ -2272,13 +2274,13 @@ const SdrObject *SwOrderIter::Top() if ( pPage->GetSortedObjs() ) { sal_uInt32 nTopOrd = 0; - const SwSortedObjs *pObjs = pPage->GetSortedObjs(); + const SwSortedObjs *pObjs = pPage->GetSortedObjs(); if ( pObjs->Count() ) { - (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! + (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! for ( sal_uInt16 i = 0; i < pObjs->Count(); ++i ) { - const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); + const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); if ( bFlysOnly && !pObj->ISA(SwVirtFlyDrawObj) ) continue; sal_uInt32 nTmp = pObj->GetOrdNumDirect(); @@ -2301,13 +2303,13 @@ const SdrObject *SwOrderIter::Bottom() if ( pPage->GetSortedObjs() ) { sal_uInt32 nBotOrd = USHRT_MAX; - const SwSortedObjs *pObjs = pPage->GetSortedObjs(); + const SwSortedObjs *pObjs = pPage->GetSortedObjs(); if ( pObjs->Count() ) { - (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! + (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! for ( sal_uInt16 i = 0; i < pObjs->Count(); ++i ) { - const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); + const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); if ( bFlysOnly && !pObj->ISA(SwVirtFlyDrawObj) ) continue; sal_uInt32 nTmp = pObj->GetOrdNumDirect(); @@ -2334,7 +2336,7 @@ const SdrObject *SwOrderIter::Next() const SwSortedObjs *pObjs = pPage->GetSortedObjs(); if ( pObjs->Count() ) { - (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! + (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! for ( sal_uInt16 i = 0; i < pObjs->Count(); ++i ) { const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); @@ -2364,7 +2366,7 @@ const SdrObject *SwOrderIter::Prev() const SwSortedObjs *pObjs = pPage->GetSortedObjs(); if ( pObjs->Count() ) { - (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! + (*pObjs)[0]->GetDrawObj()->GetOrdNum(); //Aktualisieren erzwingen! for ( sal_uInt16 i = 0; i < pObjs->Count(); ++i ) { const SdrObject* pObj = (*pObjs)[i]->GetDrawObj(); @@ -2629,7 +2631,7 @@ void RestoreCntnt( SwFrm *pSav, SwLayoutFrm *pParent, SwFrm *pSibling, bool bGro SWRECTFN( pParent ) //Wenn es bereits FlowFrms unterhalb des neuen Parent gibt, so wird die - //Kette, beginnend mit pSav, hinter dem letzten angehaengt. + //Kette, beginnend mit pSav, hinter dem letzten angehaengt. //Die Teile werden kurzerhand insertet und geeignet invalidiert. //Unterwegs werden die Flys der CntntFrms bei der Seite angemeldet. @@ -2638,7 +2640,7 @@ void RestoreCntnt( SwFrm *pSav, SwLayoutFrm *pParent, SwFrm *pSibling, bool bGro if ( pPage ) pPage->InvalidatePage( pPage ); //Invalides Layout anmelden. - //Vorgaenger festellen und die Verbindung herstellen bzw. initialisieren. + //Vorgaenger feststellen und die Verbindung herstellen bzw. initialisieren. pSav->pPrev = pSibling; SwFrm* pNxt; if ( pSibling ) @@ -2709,8 +2711,8 @@ void RestoreCntnt( SwFrm *pSav, SwLayoutFrm *pParent, SwFrm *pSibling, bool bGro pNxt->pPrev = pLast; } - if ( bGrow ) - pParent->Grow( nGrowVal ); + if ( bGrow ) + pParent->Grow( nGrowVal ); } /************************************************************************* @@ -2845,7 +2847,7 @@ void RegistFlys( SwPageFrm *pPage, const SwLayoutFrm *pLay ) |* void Notify() |* |* Beschreibung Benachrichtigt den Hintergrund je nach der -|* Veraenderung zwischen altem und neuem Rechteckt. +|* Veraenderung zwischen altem und neuem Rechteck. |* |*************************************************************************/ @@ -3146,8 +3148,8 @@ void Notify_Background( const SdrObject* pObj, pFlyFrm && !pFly->IsLowerOf( pFlyFrm ) ) { const SwFmtHoriOrient &rH = pFly->GetFmt()->GetHoriOrient(); - if ( text::HoriOrientation::NONE != rH.GetHoriOrient() && - text::HoriOrientation::CENTER != rH.GetHoriOrient() && + if ( text::HoriOrientation::NONE != rH.GetHoriOrient() && + text::HoriOrientation::CENTER != rH.GetHoriOrient() && ( !pFly->IsAutoPos() || text::RelOrientation::CHAR != rH.GetRelationOrient() ) && (pFly->Frm().Bottom() >= rRect.Top() && pFly->Frm().Top() <= rRect.Bottom()) ) @@ -3362,7 +3364,7 @@ class SwFrmHolder : private SfxListener { SwFrm* pFrm; bool bSet; - virtual void Notify( SfxBroadcaster& rBC, const SfxHint& rHint ); + virtual void Notify( SfxBroadcaster& rBC, const SfxHint& rHint ); public: SwFrmHolder() : pFrm(0), bSet(false) {} void SetFrm( SwFrm* pHold ); @@ -3372,18 +3374,18 @@ public: }; void SwFrmHolder::SetFrm( SwFrm* pHold ) -{ - bSet = true; - pFrm = pHold; - StartListening(*pHold); +{ + bSet = true; + pFrm = pHold; + StartListening(*pHold); } -void SwFrmHolder::Reset() -{ - if (pFrm) - EndListening(*pFrm); - bSet = false; - pFrm = 0; +void SwFrmHolder::Reset() +{ + if (pFrm) + EndListening(*pFrm); + bSet = false; + pFrm = 0; } void SwFrmHolder::Notify( SfxBroadcaster& rBC, const SfxHint& rHint ) @@ -3442,9 +3444,9 @@ SwFrm* GetFrmOfModify( const SwRootFrm* pLayout, SwModify const& rMod, sal_uInt1 } // #127369# - // aIter.IsChanged checks if the current pTmpFrm has been deleted while + // aIter.IsChanged checks if the current pTmpFrm has been deleted while // it is the current iterator - // FrmHolder watches for deletion of the current pMinFrm + // FrmHolder watches for deletion of the current pMinFrm if( aIter.IsChanged() || ( aHolder.IsSet() && !aHolder.GetFrm() ) ) { // restart iteration @@ -3587,30 +3589,29 @@ const SwCntntFrm* GetCellCntnt( const SwLayoutFrm& rCell ) pCntnt = pCntnt->FindNextCnt(); - } - else - break; - } - return pCntnt; + } + else + break; + } + return pCntnt; } /** Can be used to check if a frame has been deleted */ bool SwDeletionChecker::HasBeenDeleted() { - if ( !mpFrm || !mpRegIn ) - return false; + if ( !mpFrm || !mpRegIn ) + return false; - SwIterator<SwFrm,SwModify> aIter(*mpRegIn); - SwFrm* pLast = aIter.First(); - while ( pLast ) - { - if ( pLast == mpFrm ) - return false; - pLast = aIter.Next(); - } + SwIterator<SwFrm,SwModify> aIter(*mpRegIn); + SwFrm* pLast = aIter.First(); + while ( pLast ) + { + if ( pLast == mpFrm ) + return false; + pLast = aIter.Next(); + } - return true; + return true; } -