[Libreoffice-commits] core.git: sw/source

Chris Laplante mostthingsweb at gmail.com
Fri Apr 18 02:23:01 PDT 2014


 sw/source/core/text/EnhancedPDFExportHelper.cxx |   69 +++---------------------
 sw/source/core/text/atrhndl.hxx                 |   16 +----
 sw/source/core/text/atrstck.cxx                 |   69 +-----------------------
 sw/source/core/text/blink.cxx                   |   10 +--
 sw/source/core/text/frmcrsr.cxx                 |   44 ---------------
 sw/source/core/text/frmform.cxx                 |   35 ++++--------
 sw/source/core/text/frminf.cxx                  |   41 --------------
 sw/source/core/text/guess.cxx                   |   11 +--
 sw/source/core/text/guess.hxx                   |    4 -
 sw/source/core/text/inftxt.cxx                  |   61 ++++++---------------
 sw/source/core/text/inftxt.hxx                  |   54 ++----------------
 sw/source/core/text/itradj.cxx                  |   61 ++-------------------
 12 files changed, 67 insertions(+), 408 deletions(-)

New commits:
commit d8f8c8aa4544e2a0e7e3c43764080d5fe0effacf
Author: Chris Laplante <mostthingsweb at gmail.com>
Date:   Fri Apr 18 00:59:07 2014 -0400

    Begin removing ASCII art and useless comments from sw
    
    Change-Id: Ic367b2ebca1a766c830ccd44a84274d86be4f080
    Reviewed-on: https://gerrit.libreoffice.org/9090
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/text/EnhancedPDFExportHelper.cxx b/sw/source/core/text/EnhancedPDFExportHelper.cxx
index 4a59b55..f310c5f 100644
--- a/sw/source/core/text/EnhancedPDFExportHelper.cxx
+++ b/sw/source/core/text/EnhancedPDFExportHelper.cxx
@@ -258,9 +258,6 @@ bool lcl_HasPreviousParaSameNumRule( const SwTxtNode& rNode )
 
 } // end namespace
 
-/*
- * SwTaggedPDFHelper::SwTaggedPDFHelper()
- */
 SwTaggedPDFHelper::SwTaggedPDFHelper( const Num_Info* pNumInfo,
                                       const Frm_Info* pFrmInfo,
                                       const Por_Info* pPorInfo,
@@ -297,9 +294,6 @@ SwTaggedPDFHelper::SwTaggedPDFHelper( const Num_Info* pNumInfo,
     }
 }
 
-/*
- * SwTaggedPDFHelper::~SwTaggedPDFHelper()
- */
 SwTaggedPDFHelper::~SwTaggedPDFHelper()
 {
     if ( mpPDFExtOutDevData && mpPDFExtOutDevData->GetIsExportTaggedPDF() )
@@ -319,9 +313,6 @@ SwTaggedPDFHelper::~SwTaggedPDFHelper()
     }
 }
 
-/*
- * SwTaggedPDFHelper::CheckReopenTag()
- */
 bool SwTaggedPDFHelper::CheckReopenTag()
 {
     bool bRet = false;
@@ -391,9 +382,6 @@ bool SwTaggedPDFHelper::CheckReopenTag()
     return bRet && !bContinue;
 }
 
-/*
- * SwTaggedPDFHelper::CheckRestoreTag()
- */
 bool SwTaggedPDFHelper::CheckRestoreTag() const
 {
     bool bRet = false;
@@ -413,9 +401,6 @@ bool SwTaggedPDFHelper::CheckRestoreTag() const
     return bRet;
 }
 
-/*
- * SwTaggedPDFHelper::BeginTag()
- */
 void SwTaggedPDFHelper::BeginTag( vcl::PDFWriter::StructElement eType, const OUString& rString )
 {
     // write new tag
@@ -474,9 +459,6 @@ void SwTaggedPDFHelper::BeginTag( vcl::PDFWriter::StructElement eType, const OUS
     SetAttributes( eType );
 }
 
-/*
- * SwTaggedPDFHelper::EndTag()
- */
 void SwTaggedPDFHelper::EndTag()
 {
     mpPDFExtOutDevData->EndStructureElement();
@@ -486,11 +468,7 @@ void SwTaggedPDFHelper::EndTag()
 #endif
 }
 
-/*
- * SwTaggedPDFHelper::SetAttributes()
- *
- * Sets the attributes according to the structure type.
- */
+// Sets the attributes according to the structure type.
 void SwTaggedPDFHelper::SetAttributes( vcl::PDFWriter::StructElement eType )
 {
     vcl::PDFWriter::StructAttributeValue eVal;
@@ -845,9 +823,6 @@ void SwTaggedPDFHelper::SetAttributes( vcl::PDFWriter::StructElement eType )
     }
 }
 
-/*
- * SwTaggedPDFHelper::BeginNumberedListStructureElements()
- */
 void SwTaggedPDFHelper::BeginNumberedListStructureElements()
 {
     OSL_ENSURE( mpNumInfo, "List without mpNumInfo?" );
@@ -985,9 +960,6 @@ void SwTaggedPDFHelper::BeginNumberedListStructureElements()
     }
 }
 
-/*
- * SwTaggedPDFHelper::BeginBlockStructureElements()
- */
 void SwTaggedPDFHelper::BeginBlockStructureElements()
 {
     const SwFrm* pFrm = &mpFrmInfo->mrFrm;
@@ -1323,9 +1295,6 @@ void SwTaggedPDFHelper::BeginBlockStructureElements()
     }
 }
 
-/*
- * SwTaggedPDFHelper::EndStructureElements()
- */
 void SwTaggedPDFHelper::EndStructureElements()
 {
     while ( nEndStructureElement > 0 )
@@ -1337,9 +1306,6 @@ void SwTaggedPDFHelper::EndStructureElements()
     CheckRestoreTag();
 }
 
-/*
- * SwTaggedPDFHelper::BeginInlineStructureElements()
- */
 void SwTaggedPDFHelper::BeginInlineStructureElements()
 {
     const SwLinePortion* pPor = &mpPorInfo->mrPor;
@@ -1471,18 +1437,12 @@ void SwTaggedPDFHelper::BeginInlineStructureElements()
     }
 }
 
-/*
- * static SwTaggedPDFHelper::IsExportTaggedPDF
- */
- bool SwTaggedPDFHelper::IsExportTaggedPDF( const OutputDevice& rOut )
- {
+bool SwTaggedPDFHelper::IsExportTaggedPDF( const OutputDevice& rOut )
+{
     vcl::PDFExtOutDevData* pPDFExtOutDevData = PTR_CAST( vcl::PDFExtOutDevData, rOut.GetExtOutDevData() );
     return pPDFExtOutDevData && pPDFExtOutDevData->GetIsExportTaggedPDF();
- }
+}
 
-/*
- * SwEnhancedPDFExportHelper::SwEnhancedPDFExportHelper()
- */
 SwEnhancedPDFExportHelper::SwEnhancedPDFExportHelper( SwEditShell& rSh,
                                                       OutputDevice& rOut,
                                                       const OUString& rPageRange,
@@ -1542,9 +1502,6 @@ SwEnhancedPDFExportHelper::~SwEnhancedPDFExportHelper()
     delete mpRangeEnum;
 }
 
-/*
- * SwEnhancedPDFExportHelper::EnhancedPDFExport()
- */
 void SwEnhancedPDFExportHelper::EnhancedPDFExport()
 {
     vcl::PDFExtOutDevData* pPDFExtOutDevData =
@@ -2106,12 +2063,8 @@ void SwEnhancedPDFExportHelper::EnhancedPDFExport()
     mrOut.Pop();
 }
 
-/*
- * SwEnhancedPDFExportHelper::CalcOutputPageNum()
- *
- * Returns the page number in the output pdf on which the given rect is located.
- * If this page is duplicated, method will return first occurrence of it.
- */
+// Returns the page number in the output pdf on which the given rect is located.
+// If this page is duplicated, method will return first occurrence of it.
 sal_Int32 SwEnhancedPDFExportHelper::CalcOutputPageNum( const SwRect& rRect ) const
 {
     std::vector< sal_Int32 > aPageNums = CalcOutputPageNums( rRect );
@@ -2120,13 +2073,9 @@ sal_Int32 SwEnhancedPDFExportHelper::CalcOutputPageNum( const SwRect& rRect ) co
     return -1;
 }
 
-/*
- * SwEnhancedPDFExportHelper::CalcOutputPageNums()
- *
- * Returns a vector of the page numbers in the output pdf on which the given
- * rect is located. There can be many such pages since StringRangeEnumerator
- * allows duplication of its entries.
- */
+// Returns a vector of the page numbers in the output pdf on which the given
+// rect is located. There can be many such pages since StringRangeEnumerator
+// allows duplication of its entries.
 std::vector< sal_Int32 > SwEnhancedPDFExportHelper::CalcOutputPageNums(
     const SwRect& rRect ) const
 {
diff --git a/sw/source/core/text/atrhndl.hxx b/sw/source/core/text/atrhndl.hxx
index 7cbbc41..ed41f2a 100644
--- a/sw/source/core/text/atrhndl.hxx
+++ b/sw/source/core/text/atrhndl.hxx
@@ -32,23 +32,13 @@ class SwViewShell;
 class SfxPoolItem;
 extern const sal_uInt8 StackPos[];
 
-/*************************************************************************
- *                      class SwAttrHandler
- *
- * Used by Attribute Iterators to organize attributes on stacks to
- * find the valid attribute in each category
- *************************************************************************/
-
+// Used by Attribute Iterators to organize attributes on stacks to
+// find the valid attribute in each category
 class SwAttrHandler
 {
 private:
 
-    /*************************************************************************
-     *                      class SwAttrStack
-     *
-     * Container for SwTxtAttr Objects
-     *************************************************************************/
-
+    // Container for SwTxtAttr Objects
     class SwAttrStack
     {
     private:
diff --git a/sw/source/core/text/atrstck.cxx b/sw/source/core/text/atrstck.cxx
index b2f628b..0de34cd 100644
--- a/sw/source/core/text/atrstck.cxx
+++ b/sw/source/core/text/atrstck.cxx
@@ -130,14 +130,10 @@ const sal_uInt8 StackPos[ static_cast<sal_uInt16>(RES_TXTATR_WITHEND_END) -
     43, // RES_TXTATR_INPUTFIELD                 // 55
 };
 
-/*************************************************************************
- *                      CharFmt::GetItem
- * returns the item set associated with an character/inet/auto style
- *************************************************************************/
-
 namespace CharFmt
 {
 
+// Returns the item set associated with an character/inet/auto style
 const SfxItemSet* GetItemSet( const SfxPoolItem& rAttr )
 {
     const SfxItemSet* pSet = 0;
@@ -161,11 +157,7 @@ const SfxItemSet* GetItemSet( const SfxPoolItem& rAttr )
     return pSet;
 }
 
-/*************************************************************************
- *                      CharFmt::GetItem
- * extracts pool item of type nWhich from rAttr
- *************************************************************************/
-
+// Extracts pool item of type nWhich from rAttr
 const SfxPoolItem* GetItem( const SwTxtAttr& rAttr, sal_uInt16 nWhich )
 {
     if ( RES_TXTATR_INETFMT == rAttr.Which() ||
@@ -185,11 +177,7 @@ const SfxPoolItem* GetItem( const SwTxtAttr& rAttr, sal_uInt16 nWhich )
     return ( nWhich == rAttr.Which() ) ? &rAttr.GetAttr() : 0;
 }
 
-/*************************************************************************
- *                      CharFmt::IsItemIncluded
- * checks if item is included in character/inet/auto style
- *************************************************************************/
-
+// checks if item is included in character/inet/auto style
 bool IsItemIncluded( const sal_uInt16 nWhich, const SwTxtAttr *pAttr )
 {
     bool bRet = false;
@@ -200,7 +188,6 @@ bool IsItemIncluded( const sal_uInt16 nWhich, const SwTxtAttr *pAttr )
 
     return bRet;
 }
-
 }
 
 /*************************************************************************
@@ -281,20 +268,12 @@ static bool lcl_ChgHyperLinkColor( const SwTxtAttr& rAttr,
     return false;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrStack::SwAttrStack()
- *************************************************************************/
-
 inline SwAttrHandler::SwAttrStack::SwAttrStack()
     : nCount( 0 ), nSize( INITIAL_NUM_ATTR )
 {
     pArray = pInitialArray;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrStack::Insert()
- *************************************************************************/
-
 void SwAttrHandler::SwAttrStack::Insert( const SwTxtAttr& rAttr, const sal_uInt16 nPos )
 {
     // do we still have enough space?
@@ -334,10 +313,6 @@ void SwAttrHandler::SwAttrStack::Insert( const SwTxtAttr& rAttr, const sal_uInt1
     nCount++;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrStack::Remove()
- *************************************************************************/
-
 void SwAttrHandler::SwAttrStack::Remove( const SwTxtAttr& rAttr )
 {
     sal_uInt16 nPos = Pos( rAttr );
@@ -350,19 +325,11 @@ void SwAttrHandler::SwAttrStack::Remove( const SwTxtAttr& rAttr )
     }
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrStack::Top()
- *************************************************************************/
-
 const SwTxtAttr* SwAttrHandler::SwAttrStack::Top() const
 {
     return nCount ? pArray[ nCount - 1 ] : 0;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrStack::Pos()
- *************************************************************************/
-
 sal_uInt16 SwAttrHandler::SwAttrStack::Pos( const SwTxtAttr& rAttr ) const
 {
     if ( ! nCount )
@@ -379,10 +346,6 @@ sal_uInt16 SwAttrHandler::SwAttrStack::Pos( const SwTxtAttr& rAttr ) const
     return USHRT_MAX;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::SwAttrHandler()
- *************************************************************************/
-
 SwAttrHandler::SwAttrHandler()
     : mpIDocumentSettingAccess(NULL)
     , mpShell(NULL)
@@ -397,10 +360,6 @@ SwAttrHandler::~SwAttrHandler()
     delete pFnt;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::Init()
- *************************************************************************/
-
 void SwAttrHandler::Init( const SwAttrSet& rAttrSet,
                           const IDocumentSettingAccess& rIDocumentSettingAcces,
                           const SwViewShell* pSh )
@@ -460,10 +419,6 @@ void SwAttrHandler::Reset( )
         aAttrStack[ i ].Reset();
 }
 
-/*************************************************************************
- *                      SwAttrHandler::PushAndChg()
- *************************************************************************/
-
 void SwAttrHandler::PushAndChg( const SwTxtAttr& rAttr, SwFont& rFnt )
 {
     // these special attributes in fact represent a collection of attributes
@@ -508,10 +463,6 @@ void SwAttrHandler::PushAndChg( const SwTxtAttr& rAttr, SwFont& rFnt )
     }
 }
 
-/*************************************************************************
- *                      SwAttrHandler::Push()
- *************************************************************************/
-
 bool SwAttrHandler::Push( const SwTxtAttr& rAttr, const SfxPoolItem& rItem )
 {
     OSL_ENSURE( rItem.Which() < RES_TXTATR_WITHEND_END,
@@ -541,10 +492,6 @@ bool SwAttrHandler::Push( const SwTxtAttr& rAttr, const SfxPoolItem& rItem )
     return false;
 }
 
-/*************************************************************************
- *                      SwAttrHandler::PopAndChg()
- *************************************************************************/
-
 void SwAttrHandler::PopAndChg( const SwTxtAttr& rAttr, SwFont& rFnt )
 {
     if ( RES_TXTATR_WITHEND_END <= rAttr.Which() )
@@ -585,12 +532,7 @@ void SwAttrHandler::PopAndChg( const SwTxtAttr& rAttr, SwFont& rFnt )
     }
 }
 
-/*************************************************************************
- *                      SwAttrHandler::Pop()
- *
- * only used during redlining
- *************************************************************************/
-
+// only used during redlining
 void SwAttrHandler::Pop( const SwTxtAttr& rAttr )
 {
     OSL_ENSURE( rAttr.Which() < RES_TXTATR_WITHEND_END,
@@ -602,9 +544,6 @@ void SwAttrHandler::Pop( const SwTxtAttr& rAttr )
     }
 }
 
-/*************************************************************************
- *                      SwAttrHandler::ActivateTop()
- *************************************************************************/
 void SwAttrHandler::ActivateTop( SwFont& rFnt, const sal_uInt16 nAttr )
 {
     OSL_ENSURE( nAttr < RES_TXTATR_WITHEND_END,
diff --git a/sw/source/core/text/blink.cxx b/sw/source/core/text/blink.cxx
index 6f16858..0555322 100644
--- a/sw/source/core/text/blink.cxx
+++ b/sw/source/core/text/blink.cxx
@@ -29,12 +29,11 @@
 // Nihct sichtbare Zeit:
 #define BLINK_OFF_TIME      800L
 
-/*************************************************************************
- * pBlink points to the instance where blinking portions need to register.
+/* pBlink points to the instance where blinking portions need to register.
  * If necessary, it needs to be created by SwBlink.
  * They are then triggered rhythimcally for a repaint. They can query
  * for being visible or invisible with IsVisible().
- *************************************************************************/
+ */
 SwBlink *pBlink = NULL;
 
 SwBlink::SwBlink()
@@ -50,12 +49,11 @@ SwBlink::~SwBlink( )
     aTimer.Stop();
 }
 
-/*************************************************************************
+/*
  * SwBlink::Blinker (timer):
  * Toggle visibility flag
  * Determine the repaint rectangle and invalidate them in their OleShells.
- *************************************************************************/
-
+ */
 IMPL_LINK_NOARG(SwBlink, Blinker)
 {
     bVisible = !bVisible;
diff --git a/sw/source/core/text/frmcrsr.cxx b/sw/source/core/text/frmcrsr.cxx
index b7040f9..4a73939 100644
--- a/sw/source/core/text/frmcrsr.cxx
+++ b/sw/source/core/text/frmcrsr.cxx
@@ -137,10 +137,6 @@ bool sw_ChangeOffset( SwTxtFrm* pFrm, sal_Int32 nNew )
     return false;
 }
 
-/*************************************************************************
- *                      GetFrmAtOfst(), GetFrmAtPos()
- *************************************************************************/
-
 SwTxtFrm& SwTxtFrm::GetFrmAtOfst( const sal_Int32 nWhere )
 {
     SwTxtFrm* pRet = this;
@@ -168,10 +164,6 @@ SwTxtFrm *SwTxtFrm::GetFrmAtPos( const SwPosition &rPos )
     return pFoll;
 }
 
-/*************************************************************************
- *                      SwTxtFrm::GetCharRect()
- *************************************************************************/
-
 /*
  * GetCharRect() returns the char's char line described by aPos.
  * GetCrsrOfst() does the reverse: It goes from a document coordinate to
@@ -357,10 +349,6 @@ bool SwTxtFrm::GetCharRect( SwRect& rOrig, const SwPosition &rPos,
     return bRet;
 }
 
-/*************************************************************************
- *                      SwTxtFrm::GetAutoPos()
- *************************************************************************/
-
 /*
  * GetAutoPos() looks up the char's char line which is described by rPos
  * and is used by the auto-positioned frame.
@@ -516,10 +504,6 @@ bool SwTxtFrm::GetTopOfLine( SwTwips& _onTopOfLine,
     return bRet;
 }
 
-/*************************************************************************
- *                      SwTxtFrm::_GetCrsrOfst()
- *************************************************************************/
-
 // Minimum distance of non-empty lines is a little less than 2 cm
 #define FILL_MIN_DIST 1100
 
@@ -666,10 +650,6 @@ bool SwTxtFrm::_GetCrsrOfst(SwPosition* pPos, const Point& rPoint,
     return true;
 }
 
-/*************************************************************************
- *                 virtual SwTxtFrm::GetCrsrOfst()
- *************************************************************************/
-
 bool SwTxtFrm::GetCrsrOfst(SwPosition* pPos, Point& rPoint,
                                SwCrsrMoveState* pCMS, bool ) const
 {
@@ -685,10 +665,6 @@ bool SwTxtFrm::GetCrsrOfst(SwPosition* pPos, Point& rPoint,
     return _GetCrsrOfst( pPos, rPoint, nChgFrm != 0, pCMS );
 }
 
-/*************************************************************************
- *                      SwTxtFrm::LeftMargin()
- *************************************************************************/
-
 /*
  * Layout-oriented cursor movement to the line start.
  */
@@ -722,10 +698,6 @@ bool SwTxtFrm::LeftMargin(SwPaM *pPam) const
     return true;
 }
 
-/*************************************************************************
- *                      SwTxtFrm::RightMargin()
- *************************************************************************/
-
 /*
  * To the line end: That's the position before the last char of the line.
  * Exception: In the last line, it should be able to place the cursor after
@@ -767,10 +739,6 @@ bool SwTxtFrm::RightMargin(SwPaM *pPam, bool bAPI) const
     return true;
 }
 
-/*************************************************************************
- *                      SwTxtFrm::_UnitUp()
- *************************************************************************/
-
 // The following two methods try to put the Crsr into the next/succsessive
 // line. If we do not have a preceding/successive line we forward the call
 // to the base class.
@@ -1174,10 +1142,6 @@ void SwTxtFrm::PrepareVisualMove( sal_Int32& nPos, sal_uInt8& nCrsrLevel,
     ubidi_close( pBidi );
 }
 
-/*************************************************************************
- *                      SwTxtFrm::_UnitDown()
- *************************************************************************/
-
 bool SwTxtFrm::_UnitDown(SwPaM *pPam, const SwTwips nOffset,
                          bool bSetInReadOnly ) const
 {
@@ -1303,10 +1267,6 @@ bool SwTxtFrm::_UnitDown(SwPaM *pPam, const SwTwips nOffset,
     return SwCntntFrm::UnitDown( pPam, nOffset, bSetInReadOnly );
 }
 
-/*************************************************************************
- *                   virtual SwTxtFrm::UnitUp()
- *************************************************************************/
-
 bool SwTxtFrm::UnitUp(SwPaM *pPam, const SwTwips nOffset,
                       bool bSetInReadOnly ) const
 {
@@ -1326,10 +1286,6 @@ bool SwTxtFrm::UnitUp(SwPaM *pPam, const SwTwips nOffset,
     return bRet;
 }
 
-/*************************************************************************
- *                   virtual SwTxtFrm::UnitDown()
- *************************************************************************/
-
 bool SwTxtFrm::UnitDown(SwPaM *pPam, const SwTwips nOffset,
                         bool bSetInReadOnly ) const
 {
diff --git a/sw/source/core/text/frmform.cxx b/sw/source/core/text/frmform.cxx
index cbabc00..d3992a4b 100644
--- a/sw/source/core/text/frmform.cxx
+++ b/sw/source/core/text/frmform.cxx
@@ -109,13 +109,9 @@ void SwTxtFrm::ValidateFrm()
     UNDO_SWAP( this )
 }
 
-/*************************************************************************
- * ValidateBodyFrm()
- * After a RemoveFtn the BodyFrm and all Frms contained within it, need to be
- * recalculated, so that the DeadLine is right.
- * First we search outwards, on the way back we calculate everything.
-*************************************************************************/
-
+// After a RemoveFtn the BodyFrm and all Frms contained within it, need to be
+// recalculated, so that the DeadLine is right.
+// First we search outwards, on the way back we calculate everything.
 void _ValidateBodyFrm( SwFrm *pFrm )
 {
     if( pFrm && !pFrm->IsCellFrm() )
@@ -462,12 +458,10 @@ void SwTxtFrm::AdjustFrm( const SwTwips nChgHght, bool bHasToFit )
             nRstHeight += nAdd;
         }
 
-/* ------------------------------------
- * nRstHeight < 0 means that the TxtFrm is located completely outside of its Upper.
- * This can happen, if it's located within a FlyAtCntFrm, which changed sides by a
- * Grow(). In such a case, it's wrong to execute the following Grow().
- * In the case of a bug, we end up with an infinite loop.
- * -----------------------------------*/
+        // nRstHeight < 0 means that the TxtFrm is located completely outside of its Upper.
+        // This can happen, if it's located within a FlyAtCntFrm, which changed sides by a
+        // Grow(). In such a case, it's wrong to execute the following Grow().
+        // In the case of a bug, we end up with an infinite loop.
         SwTwips nFrmHeight = (Frm().*fnRect->fnGetHeight)();
         SwTwips nPrtHeight = (Prt().*fnRect->fnGetHeight)();
 
@@ -544,16 +538,11 @@ com::sun::star::uno::Sequence< ::com::sun::star::style::TabStop > SwTxtFrm::GetT
     return tabs;
 }
 
-/*************************************************************************
- * SwTxtFrm::AdjustFollow()
- * AdjustFollow expects the following situation:
- * The SwTxtIter points to the lower end of the Master, the Offset is set in
- * the Follow.
- * nOffset holds the Offset in the text string, from which the Master closes
- * and the Follow starts.
- * If it's 0, the FollowFrame is deleted.
- *************************************************************************/
-
+// AdjustFollow expects the following situation:
+// The SwTxtIter points to the lower end of the Master, the Offset is set in the Follow.
+// nOffset holds the Offset in the text string, from which the Master closes
+// and the Follow starts.
+// If it's 0, the FollowFrame is deleted.
 void SwTxtFrm::_AdjustFollow( SwTxtFormatter &rLine,
                              const sal_Int32 nOffset, const sal_Int32 nEnd,
                              const sal_uInt8 nMode )
diff --git a/sw/source/core/text/frminf.cxx b/sw/source/core/text/frminf.cxx
index fbf317f..fe7aea5 100644
--- a/sw/source/core/text/frminf.cxx
+++ b/sw/source/core/text/frminf.cxx
@@ -21,10 +21,6 @@
 #include <frminf.hxx>
 #include <itrtxt.hxx>
 
-/*************************************************************************
- *                   SwTxtMargin::GetTxtStart()
- *************************************************************************/
-
 sal_Int32 SwTxtMargin::GetTxtStart() const
 {
     const OUString &rTxt = GetInfo().GetTxt();
@@ -39,10 +35,6 @@ sal_Int32 SwTxtMargin::GetTxtStart() const
     return nEnd;
 }
 
-/*************************************************************************
- *                   SwTxtMargin::GetTxtEnd()
- *************************************************************************/
-
 sal_Int32 SwTxtMargin::GetTxtEnd() const
 {
     const OUString &rTxt = GetInfo().GetTxt();
@@ -56,10 +48,6 @@ sal_Int32 SwTxtMargin::GetTxtEnd() const
     return nStart;
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::IsOneLine()
- *************************************************************************/
-
 // Does the paragraph fit into one line?
 bool SwTxtFrmInfo::IsOneLine() const
 {
@@ -81,10 +69,6 @@ bool SwTxtFrmInfo::IsOneLine() const
     return true;
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::IsFilled()
- *************************************************************************/
-
 // Is the line filled for X percent?
 bool SwTxtFrmInfo::IsFilled( const sal_uInt8 nPercent ) const
 {
@@ -98,10 +82,6 @@ bool SwTxtFrmInfo::IsFilled( const sal_uInt8 nPercent ) const
     return KSHORT(nWidth) <= pLay->Width();
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::GetLineStart()
- *************************************************************************/
-
 // Where does the text start (without whitespace)? (document global)
 SwTwips SwTxtFrmInfo::GetLineStart( const SwTxtCursor &rLine ) const
 {
@@ -116,10 +96,6 @@ SwTwips SwTxtFrmInfo::GetLineStart( const SwTxtCursor &rLine ) const
     return rLine.GetLineStart();
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::GetLineStart()
- *************************************************************************/
-
 // Where does the text start (without whitespace)? (relative in the Frame)
 SwTwips SwTxtFrmInfo::GetLineStart() const
 {
@@ -165,10 +141,6 @@ SwTwips SwTxtFrmInfo::GetCharPos( sal_Int32 nChar, bool bCenter ) const
     return (( nNext + nStt ) / 2 ) - (pFrm->Frm().*fnRect->fnGetLeft)();
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::GetSpaces()
- *************************************************************************/
-
 SwPaM *AddPam( SwPaM *pPam, const SwTxtFrm* pTxtFrm,
                 const sal_Int32 nPos, const sal_Int32 nLen )
 {
@@ -233,10 +205,6 @@ void SwTxtFrmInfo::GetSpaces( SwPaM &rPam, bool bWithLineBreak ) const
     while( aLine.Next() );
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::IsBullet()
- *************************************************************************/
-
 // Is there a bullet/symbol etc. at the text position?
 // Fonts: CharSet, SYMBOL und DONTKNOW
 bool SwTxtFrmInfo::IsBullet( sal_Int32 nTxtStart ) const
@@ -247,15 +215,10 @@ bool SwTxtFrmInfo::IsBullet( sal_Int32 nTxtStart ) const
     return aLine.IsSymbol( nTxtStart );
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::GetFirstIndent()
- *************************************************************************/
-
 // Get first line indent
 // The precondition for a positive or negative first line indent:
 // All lines (except for the first one) have the same left margin.
 // We do not want to be so picky and work with a tolerance of TOLERANCE twips.
-
 SwTwips SwTxtFrmInfo::GetFirstIndent() const
 {
     SwTxtSizeInfo aInf( (SwTxtFrm*)pFrm );
@@ -288,10 +251,6 @@ SwTwips SwTxtFrmInfo::GetFirstIndent() const
     return 1;
 }
 
-/*************************************************************************
- *                   SwTxtFrmInfo::GetBigIndent()
- *************************************************************************/
-
 sal_Int32 SwTxtFrmInfo::GetBigIndent( sal_Int32& rFndPos,
                                     const SwTxtFrm *pNextFrm ) const
 {
diff --git a/sw/source/core/text/guess.cxx b/sw/source/core/text/guess.cxx
index 21b7121..1a65e03 100644
--- a/sw/source/core/text/guess.cxx
+++ b/sw/source/core/text/guess.cxx
@@ -42,14 +42,9 @@ using namespace ::com::sun::star::linguistic2;
 
 #define CH_FULL_BLANK 0x3000
 
-/*************************************************************************
- *                      SwTxtGuess::Guess
- *
- * provides information for line break calculation
- * returns true if no line break has to be performed
- * otherwise possible break or hyphenation position is determined
- *************************************************************************/
-
+// provides information for line break calculation
+// returns true if no line break has to be performed
+// otherwise possible break or hyphenation position is determined
 bool SwTxtGuess::Guess( const SwTxtPortion& rPor, SwTxtFormatInfo &rInf,
                             const KSHORT nPorHeight )
 {
diff --git a/sw/source/core/text/guess.hxx b/sw/source/core/text/guess.hxx
index dd6153f..3b489d9 100644
--- a/sw/source/core/text/guess.hxx
+++ b/sw/source/core/text/guess.hxx
@@ -26,10 +26,6 @@
 
 class SwTxtFormatInfo;
 
-/*************************************************************************
- *                      class SwTxtGuess
- *************************************************************************/
-
 class SwTxtGuess
 {
     ::com::sun::star::uno::Reference< ::com::sun::star::linguistic2::XHyphenatedWord >  xHyphWord;
diff --git a/sw/source/core/text/inftxt.cxx b/sw/source/core/text/inftxt.cxx
index 8c303a0..d19f439 100644
--- a/sw/source/core/text/inftxt.cxx
+++ b/sw/source/core/text/inftxt.cxx
@@ -496,12 +496,7 @@ SwTxtPaintInfo::SwTxtPaintInfo( const SwTxtPaintInfo &rInf )
 
 extern Color aGlobalRetoucheColor;
 
-/*************************************************************************
- * lcl_IsDarkBackground
- *
- * Returns if the current background color is dark.
- *************************************************************************/
-
+// Returns if the current background color is dark.
 static bool lcl_IsDarkBackground( const SwTxtPaintInfo& rInf )
 {
     const Color* pCol = rInf.GetFont()->GetBackColor();
@@ -536,10 +531,6 @@ static bool lcl_IsDarkBackground( const SwTxtPaintInfo& rInf )
     return pCol->IsDark();
 }
 
-/*************************************************************************
- * SwTxtPaintInfo::_DrawText()
- *************************************************************************/
-
 void SwTxtPaintInfo::_DrawText( const OUString &rText, const SwLinePortion &rPor,
                                 const sal_Int32 nStart, const sal_Int32 nLength,
                                 const bool bKern, const bool bWrong,
@@ -788,17 +779,12 @@ void SwTxtPaintInfo::CalcRect( const SwLinePortion& rPor,
     }
 }
 
-/*************************************************************************
- * lcl_DrawSpecial
- *
- * Draws a special portion, e.g., line break portion, tab portion.
- * rPor     - The portion
- * rRect    - The rectangle surrounding the character
- * rCol     - Specify a color for the character
- * bCenter  - Draw the character centered, otherwise left aligned
- * bRotate  - Rotate the character if character rotation is set
- *************************************************************************/
-
+// Draws a special portion, e.g., line break portion, tab portion.
+// rPor     - The portion
+// rRect    - The rectangle surrounding the character
+// rCol     - Specify a color for the character
+// bCenter  - Draw the character centered, otherwise left aligned
+// bRotate  - Rotate the character if character rotation is set
 static void lcl_DrawSpecial( const SwTxtPaintInfo& rInf, const SwLinePortion& rPor,
                       SwRect& rRect, const Color& rCol, sal_Unicode cChar,
                       sal_uInt8 nOptions )
@@ -1377,14 +1363,11 @@ void SwTxtFormatInfo::CtorInitTxtFormatInfo( SwTxtFrm *pNewFrm, const bool bNewI
     Init();
 }
 
-/*************************************************************************
- * SwTxtFormatInfo::IsHyphenate()
- * If the Hyphenator returns ERROR or the language is set to NOLANGUAGE
- * we do not hyphenate.
- * Else, we always hyphenate if we do interactive hyphenation.
- * If we do not do interactive hyphenation, we only hyphenate if ParaFmt is
- * set to automatic hyphenation.
- *************************************************************************/
+// If the Hyphenator returns ERROR or the language is set to NOLANGUAGE
+// we do not hyphenate.
+// Else, we always hyphenate if we do interactive hyphenation.
+// If we do not do interactive hyphenation, we only hyphenate if ParaFmt is
+// set to automatic hyphenation.
 bool SwTxtFormatInfo::IsHyphenate() const
 {
     if( !bInterHyph && !bAutoHyph )
@@ -1404,11 +1387,6 @@ bool SwTxtFormatInfo::IsHyphenate() const
     return xHyph->hasLocale( g_pBreakIt->GetLocale(eTmp) );
 }
 
-/*************************************************************************
- * SwTxtFormatInfo::GetDropFmt()
- * Dropcaps called by the SwTxtFormatter::CTOR
-*************************************************************************/
-
 const SwFmtDrop *SwTxtFormatInfo::GetDropFmt() const
 {
     const SwFmtDrop *pDrop = &GetTxtFrm()->GetTxtNode()->GetSwAttrSet().GetDrop();
@@ -1455,15 +1433,12 @@ void SwTxtFormatInfo::Init()
     SetPaintOfst(0);
 }
 
-/*--------------------------------------------------
- * There are a few differences between a copy constructor
- * and the following constructor for multi-line formatting.
- * The root is the first line inside the multi-portion,
- * the line start is the actual position in the text,
- * the line width is the rest width from the surrounding line
- * and the bMulti and bFirstMulti-flag has to be set correctly.
- * --------------------------------------------------*/
-
+// There are a few differences between a copy constructor
+// and the following constructor for multi-line formatting.
+// The root is the first line inside the multi-portion,
+// the line start is the actual position in the text,
+// the line width is the rest width from the surrounding line
+// and the bMulti and bFirstMulti-flag has to be set correctly.
 SwTxtFormatInfo::SwTxtFormatInfo( const SwTxtFormatInfo& rInf,
     SwLineLayout& rLay, SwTwips nActWidth ) : SwTxtPaintInfo( rInf ),
     bTabOverflow( false )
diff --git a/sw/source/core/text/inftxt.hxx b/sw/source/core/text/inftxt.hxx
index 35c783e..a757222 100644
--- a/sw/source/core/text/inftxt.hxx
+++ b/sw/source/core/text/inftxt.hxx
@@ -70,11 +70,7 @@ class SwWrongList;
 #define OPTDBG( rInf )   false
 #endif
 
-/*************************************************************************
- * class SwLineInfo
- * Respects the attribute LineSpace when calculating the Height/Ascent
- *************************************************************************/
-
+// Respects the attribute LineSpace when calculating the Height/Ascent
 class SwLineInfo
 {
     friend class SwTxtIter;
@@ -122,10 +118,6 @@ public:
     friend SvStream & WriteSwLineInfo( SvStream &rOS, const SwLineInfo &rInf );
 };
 
-/*************************************************************************
- * class SwTxtInfo
- *************************************************************************/
-
 class SwTxtInfo
 {
     // Implementation in txthyph.cxx
@@ -149,10 +141,6 @@ public:
     friend SvStream & WriteSwTxtInfo( SvStream &rOS, const SwTxtInfo &rInf );
 };
 
-/*************************************************************************
- * class SwTxtSizeInfo
- *************************************************************************/
-
 typedef ::std::map< sal_uLong, sal_IntPtr > SwTxtPortionMap;
 
 class SwTxtSizeInfo : public SwTxtInfo
@@ -302,8 +290,6 @@ public:
 
     KSHORT      GetTxtHeight() const;
 
-    // GetTxtSize
-
     SwPosSize GetTxtSize( OutputDevice* pOut, const SwScriptInfo* pSI,
                           const OUString& rTxt, const sal_Int32 nIdx,
                           const sal_Int32 nLen, const sal_uInt16 nComp ) const;
@@ -315,8 +301,6 @@ public:
                                  const sal_Int32 nLen, const sal_uInt16 nComp ) const;
     inline SwPosSize GetTxtSize( const OUString &rTxt ) const;
 
-    // GetTxtBreak
-
     sal_Int32 GetTxtBreak( const long nLineWidth,
                                            const sal_Int32 nMaxLen,
                                            const sal_uInt16 nComp ) const;
@@ -393,10 +377,6 @@ public:
 #endif
 };
 
-/*************************************************************************
- * class SwTxtPaintInfo
- *************************************************************************/
-
 class SwTxtPaintInfo : public SwTxtSizeInfo
 {
     const SwWrongList *pWrongList;
@@ -535,10 +515,6 @@ public:
     inline const SwWrongList* GetSmartTags() const { return pSmartTags; }
 };
 
-/*************************************************************************
- * class SwTxtFormatInfo
- *************************************************************************/
-
 class SwTxtFormatInfo : public SwTxtPaintInfo
 {
     // temporary arguments for hyphenation
@@ -753,14 +729,11 @@ public:
     inline bool IsTabOverflow() { return bTabOverflow; }
 };
 
-/*************************************************************************
- * class SwTxtSlot
- * For the text replacement and restoration of SwTxtSizeInfo.
- * The way this is done is a bit of a hack: Although rInf is const we change it
- * anyway.
- * Because rInf is restorated again in the DTOR, we can do this.
- * You could call it a "logical const", if you wish.
-*************************************************************************/
+// For the text replacement and restoration of SwTxtSizeInfo.
+// The way this is done is a bit of a hack: Although rInf is const we change it
+// anyway.
+// Because rInf is restorated again in the DTOR, we can do this.
+// You could call it a "logical const", if you wish.
 class SwTxtSlot
 {
     OUString aTxt;
@@ -782,10 +755,6 @@ public:
     bool IsOn() const { return bOn; }
 };
 
-/*************************************************************************
- * class SwFontSave
- *************************************************************************/
-
 class SwFontSave
 {
     SwTxtSizeInfo *pInf;
@@ -797,9 +766,6 @@ public:
    ~SwFontSave();
 };
 
-/*************************************************************************
- * Inline implementations of SwTxtSizeInfo
- *************************************************************************/
 inline KSHORT SwTxtSizeInfo::GetAscent() const
 {
     SAL_WARN_IF( !GetOut(), "sw.core", "SwTxtSizeInfo::GetAscent() without m_pOut" );
@@ -827,10 +793,6 @@ inline SwPosSize SwTxtSizeInfo::GetTxtSize( const SwScriptInfo* pSI,
     return GetTxtSize( m_pOut, pSI, *m_pTxt, nNewIdx, nNewLen, nCompress );
 }
 
-/*************************************************************************
- * Inline implementations of SwTxtPaintInfo
- *************************************************************************/
-
 inline SwTwips SwTxtPaintInfo::GetPaintOfst() const
 {
     return GetParaPortion()->GetRepaint()->GetOfst();
@@ -865,10 +827,6 @@ inline void SwTxtPaintInfo::DrawMarkedText( const SwLinePortion &rPor,
     ((SwTxtPaintInfo*)this)->_DrawText( *m_pTxt, rPor, m_nIdx, nLength, bKern, bWrong, bSmartTags, bGrammarCheck );
 }
 
-/*************************************************************************
- * Inline implementations of SwTxtFormatInfo
- *************************************************************************/
-
 inline sal_Int32 SwTxtFormatInfo::GetReformatStart() const
 {
     return GetParaPortion()->GetReformat()->Start();
diff --git a/sw/source/core/text/itradj.cxx b/sw/source/core/text/itradj.cxx
index 2dca318..24e7111 100644
--- a/sw/source/core/text/itradj.cxx
+++ b/sw/source/core/text/itradj.cxx
@@ -35,10 +35,6 @@
 
 using namespace ::com::sun::star;
 
-/*************************************************************************
- * SwTxtAdjuster::FormatBlock()
- *************************************************************************/
-
 void SwTxtAdjuster::FormatBlock( )
 {
     // Block format does not apply to the last line.
@@ -113,9 +109,6 @@ void SwTxtAdjuster::FormatBlock( )
     GetInfo().GetParaPortion()->GetRepaint()->SetOfst(0);
 }
 
-/*************************************************************************
- * lcl_CheckKashidaPositions()
- *************************************************************************/
 static bool lcl_CheckKashidaPositions( SwScriptInfo& rSI, SwTxtSizeInfo& rInf, SwTxtIter& rItr,
                                 sal_Int32& rKashidas, sal_Int32& nGluePortion )
 {
@@ -195,9 +188,6 @@ static bool lcl_CheckKashidaPositions( SwScriptInfo& rSI, SwTxtSizeInfo& rInf, S
     return (rKashidas > 0);
 }
 
-/*************************************************************************
- * lcl_CheckKashidaWidth()
- *************************************************************************/
 static bool lcl_CheckKashidaWidth ( SwScriptInfo& rSI, SwTxtSizeInfo& rInf, SwTxtIter& rItr, sal_Int32& rKashidas,
                              sal_Int32& nGluePortion, const long nGluePortionWidth, long& nSpaceAdd )
 {
@@ -255,14 +245,9 @@ static bool lcl_CheckKashidaWidth ( SwScriptInfo& rSI, SwTxtSizeInfo& rInf, SwTx
    return true;
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcNewBlock()
- *
- * CalcNewBlock() must only be called _after_ CalcLine()!
- * We always span between two RandPortions or FixPortions (Tabs and Flys).
- * We count the Glues and call ExpandBlock.
- *************************************************************************/
-
+// CalcNewBlock() must only be called _after_ CalcLine()!
+// We always span between two RandPortions or FixPortions (Tabs and Flys).
+// We count the Glues and call ExpandBlock.
 void SwTxtAdjuster::CalcNewBlock( SwLineLayout *pCurrent,
                                   const SwLinePortion *pStopAt, SwTwips nReal, bool bSkipKashida )
 {
@@ -410,10 +395,6 @@ void SwTxtAdjuster::CalcNewBlock( SwLineLayout *pCurrent,
     }
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcKanaAdj()
- *************************************************************************/
-
 SwTwips SwTxtAdjuster::CalcKanaAdj( SwLineLayout* pCurrent )
 {
     OSL_ENSURE( pCurrent->Height(), "SwTxtAdjuster::CalcBlockAdjust: missing CalcLine()" );
@@ -556,10 +537,6 @@ SwTwips SwTxtAdjuster::CalcKanaAdj( SwLineLayout* pCurrent )
     return nRepaintOfst;
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcRightMargin()
- *************************************************************************/
-
 SwMarginPortion *SwTxtAdjuster::CalcRightMargin( SwLineLayout *pCurrent,
     SwTwips nReal )
 {
@@ -611,10 +588,6 @@ SwMarginPortion *SwTxtAdjuster::CalcRightMargin( SwLineLayout *pCurrent,
     return pRight;
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcFlyAdjust()
- *************************************************************************/
-
 void SwTxtAdjuster::CalcFlyAdjust( SwLineLayout *pCurrent )
 {
     // 1) We insert a left margin:
@@ -690,10 +663,6 @@ void SwTxtAdjuster::CalcFlyAdjust( SwLineLayout *pCurrent )
         pLeft->AdjustRight( pCurrent );
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcAdjLine()
- *************************************************************************/
-
 void SwTxtAdjuster::CalcAdjLine( SwLineLayout *pCurrent )
 {
     OSL_ENSURE( pCurrent->IsFormatAdj(), "CalcAdjLine: Why?" );
@@ -720,15 +689,10 @@ void SwTxtAdjuster::CalcAdjLine( SwLineLayout *pCurrent )
     }
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcFlyPortion()
- *
- * This is a quite complicated calculation: nCurrWidth is the width _before_
- * adding the word, that still fits onto the line! For this reason the FlyPortion's
- * width is still correct if we get a deadlock-situation of:
- * bFirstWord && !WORDFITS
- *************************************************************************/
-
+// This is a quite complicated calculation: nCurrWidth is the width _before_
+// adding the word, that still fits onto the line! For this reason the FlyPortion's
+// width is still correct if we get a deadlock-situation of:
+// bFirstWord && !WORDFITS
 SwFlyPortion *SwTxtAdjuster::CalcFlyPortion( const long nRealWidth,
                                              const SwRect &rCurrRect )
 {
@@ -773,12 +737,7 @@ SwFlyPortion *SwTxtAdjuster::CalcFlyPortion( const long nRealWidth,
     return pFlyPortion;
 }
 
-/*************************************************************************
- * SwTxtPainter::_CalcDropAdjust()
- * Drops and Adjustment
- * CalcDropAdjust is called at the end by Format() if needed
- *************************************************************************/
-
+// CalcDropAdjust is called at the end by Format() if needed
 void SwTxtAdjuster::CalcDropAdjust()
 {
     OSL_ENSURE( 1<GetDropLines() && SVX_ADJUST_LEFT!=GetAdjust() && SVX_ADJUST_BLOCK!=GetAdjust(),
@@ -862,10 +821,6 @@ void SwTxtAdjuster::CalcDropAdjust()
     }
 }
 
-/*************************************************************************
- * SwTxtAdjuster::CalcDropRepaint()
- *************************************************************************/
-
 void SwTxtAdjuster::CalcDropRepaint()
 {
     Top();


More information about the Libreoffice-commits mailing list