You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openoffice.apache.org by ms...@apache.org on 2021/08/24 12:23:55 UTC

[openoffice] branch AOO42X updated: Fixed typos, small cleanup

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 <ms...@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;
 }
 
-