[Libreoffice-commits] core.git: sw/source
Muhammet Kara
muhammet.kara at pardus.org.tr
Fri Apr 1 06:25:01 UTC 2016
sw/source/core/text/frmform.cxx | 33 ++++++++++++++++-----------------
1 file changed, 16 insertions(+), 17 deletions(-)
New commits:
commit 403793990c1258db5b2bc4f709e6028a3d5ea2cb
Author: Muhammet Kara <muhammet.kara at pardus.org.tr>
Date: Thu Mar 31 11:26:34 2016 +0300
tdf#90834 Cleanup in-line version control history comments
Turn in-line version control history comments into
meaningful good comments: frmform.cxx
Change-Id: Iadfeca748b7d1f0a8e5eadb684ebf66d8e026931
Reviewed-on: https://gerrit.libreoffice.org/23705
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: jan iversen <jani at documentfoundation.org>
Tested-by: jan iversen <jani at documentfoundation.org>
diff --git a/sw/source/core/text/frmform.cxx b/sw/source/core/text/frmform.cxx
index 12c7595..d596535 100644
--- a/sw/source/core/text/frmform.cxx
+++ b/sw/source/core/text/frmform.cxx
@@ -250,10 +250,10 @@ bool SwTextFrame::CalcFollow( const sal_Int32 nTextOfst )
( ! pSct->IsVertical() && !pSct->Frame().Height() ) )
break;
}
- // OD 14.03.2003 #i11760# - intrinsic format of follow is controlled.
+ // i#11760 - Intrinsic format of follow is controlled.
if ( FollowFormatAllowed() )
{
- // OD 14.03.2003 #i11760# - no nested format of follows, if
+ // i#11760 - No nested format of follows, if
// text frame is contained in a column frame.
// Thus, forbid intrinsic format of follow.
{
@@ -289,7 +289,7 @@ bool SwTextFrame::CalcFollow( const sal_Int32 nTextOfst )
OSL_ENSURE( !pMyFollow->GetPrev(), "SwTextFrame::CalcFollow: very cheesy follow" );
}
- // OD 14.03.2003 #i11760# - reset control flag for follow format.
+ // i#11760 - Reset control flag for follow format.
pMyFollow->AllowFollowFormat();
}
@@ -405,8 +405,7 @@ void SwTextFrame::AdjustFrame( const SwTwips nChgHght, bool bHasToFit )
MakePos();
if ( aOldPos != Frame().Pos() )
{
- // OD 2004-07-01 #i28701# - use new method <SwFrame::InvalidateObjs(..)>
- // No format is performed for the floating screen objects.
+ // i#28701 - No format is performed for the floating screen objects.
InvalidateObjs();
}
}
@@ -439,8 +438,7 @@ void SwTextFrame::AdjustFrame( const SwTwips nChgHght, bool bHasToFit )
// We can get a bit of space in table cells, because there could be some
// left through a vertical alignment to the top.
- // #115759# - assure, that first lower in upper
- // is the current one or is valid.
+ // Assure that first lower in upper is the current one or is valid.
if ( IsInTab() &&
( GetUpper()->Lower() == this ||
GetUpper()->Lower()->IsValid() ) )
@@ -642,8 +640,8 @@ SwContentFrame *SwTextFrame::JoinFrame()
pFoll->MoveFlyInCnt( this, nStart, COMPLETE_STRING );
pFoll->SetFootnote( false );
- // #i27138#
- // notify accessibility paragraphs objects about changed CONTENT_FLOWS_FROM/_TO relation.
+ // i#27138
+ // Notify accessibility paragraphs objects about changed CONTENT_FLOWS_FROM/_TO relation.
// Relation CONTENT_FLOWS_FROM for current next paragraph will change
// and relation CONTENT_FLOWS_TO for current previous paragraph, which
// is <this>, will change.
@@ -676,7 +674,7 @@ void SwTextFrame::SplitFrame( const sal_Int32 nTextPos )
SetFollow( pNew );
pNew->Paste( GetUpper(), GetNext() );
- // #i27138#
+ // i#27138
// notify accessibility paragraphs objects about changed CONTENT_FLOWS_FROM/_TO relation.
// Relation CONTENT_FLOWS_FROM for current next paragraph will change
// and relation CONTENT_FLOWS_TO for current previous paragraph, which
@@ -970,7 +968,7 @@ void SwTextFrame::FormatAdjust( SwTextFormatter &rLine,
!rFrameBreak.IsInside( rLine ) )
: rFrameBreak.IsBreakNow( rLine ) ) ) )
? 1 : 0;
- // --> OD #i84870#
+ // i#84870
// no split of text frame, which only contains a as-character anchored object
bool bOnlyContainsAsCharAnchoredObj =
!IsFollow() && nStrLen == 1 &&
@@ -985,7 +983,7 @@ void SwTextFrame::FormatAdjust( SwTextFormatter &rLine,
{
nNew = 0;
}
- // <--
+
if ( nNew )
{
SplitFrame( nEnd );
@@ -1062,7 +1060,7 @@ void SwTextFrame::FormatAdjust( SwTextFormatter &rLine,
{
// Only split frame, if the frame contains
// content or contains no content, but has a numbering.
- // OD #i84870# - no split, if text frame only contains one
+ // i#84870 - No split, if text frame only contains one
// as-character anchored object.
if ( !bOnlyContainsAsCharAnchoredObj &&
( nStrLen > 0 ||
@@ -1290,7 +1288,7 @@ void SwTextFrame::_Format( SwTextFormatter &rLine, SwTextFormatInfo &rInf,
// Unfortunately it can: Text size changes + FlyFrames.
// The backlash can affect multiple lines (Frame!)!
- // #i46560#
+ // i#46560
// FME: Yes, consider this case: (word) has to go to the next line
// because) is a forbidden character at the beginning of a line although
// (word would still fit on the previous line. Adding text right in front
@@ -1299,7 +1297,7 @@ void SwTextFrame::_Format( SwTextFormatter &rLine, SwTextFormatInfo &rInf,
// nevertheless it should be sufficient.
bool bPrev = rLine.GetPrev() &&
( FindBrk( rString, rLine.GetStart(), rReformat.Start() + 1 )
- // #i46560#
+ // i#46560
+ 1
>= rReformat.Start() ||
rLine.GetCurr()->IsRest() );
@@ -1829,7 +1827,7 @@ void SwTextFrame::Format( vcl::RenderContext* pRenderContext, const SwBorderAttr
{
SwFrame* pPre = GetPrev();
if( pPre &&
- // #i10826# It's the first, it cannot keep!
+ // i#10826 It's the first, it cannot keep!
pPre->GetIndPrev() &&
pPre->GetAttrSet()->GetKeep().GetValue() )
{
@@ -1846,7 +1844,8 @@ void SwTextFrame::Format( vcl::RenderContext* pRenderContext, const SwBorderAttr
pPara->SetPrepMustFit( false );
CalcBaseOfstForFly();
- _CalcHeightOfLastLine(); // #i11860#
+ _CalcHeightOfLastLine(); // i#11860 - Adjust spacing implementation for
+ // object positioning - Compatibility to MS Word
}
// bForceQuickFormat is set if GetFormatted() has been called during the
More information about the Libreoffice-commits
mailing list