[Libreoffice-commits] .: 2 commits - sw/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Thu Sep 20 11:31:13 PDT 2012
sw/source/core/text/txttab.cxx | 48 ++++++------
sw/source/core/text/widorp.cxx | 152 +++++++++++++++++++----------------------
2 files changed, 97 insertions(+), 103 deletions(-)
New commits:
commit 606b57c625aebabd004a4f3e3a2552f3a03551a0
Author: Michael Stahl <mstahl at redhat.com>
Date: Thu Sep 20 20:28:00 2012 +0200
sw: tweak previous comment translation
Change-Id: Id4a89bd131f37b6844046803aae9b2f6e081af0d
diff --git a/sw/source/core/text/widorp.cxx b/sw/source/core/text/widorp.cxx
index b761ce3..4fe8c41 100644
--- a/sw/source/core/text/widorp.cxx
+++ b/sw/source/core/text/widorp.cxx
@@ -94,10 +94,10 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const SwTwips nRst )
/* BP 18.6.93: Widows.
* In contrast to the first implementation the Widows are not calculated
- * in advance but detected when formating the splitted Follow.
+ * in advance but detected when formating the split Follow.
* In Master the Widows-calculation is dropped completely
* (nWidows is manipulated). If the Follow detects that the
- * Widows rule applies it sends a Prepare to its ancestor.
+ * Widows rule applies it sends a Prepare to its predecessor.
* A special problem is when the Widow rule applies but in Master
* there are some lines available.
*
@@ -113,9 +113,9 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const SwTwips nRst )
* One difficulty with Widows and different formats between
* Master- and Follow-Frame:
* Example: If the first column is 3cm and the second is 4cm and
- * Widows is set to 3. The decision if the Widows rule matches can not
+ * Widows is set to 3, the decision if the Widows rule matches can not
* be done until the Follow is formated. Unfortunately this is crucial
- * sto decide if the whole paragraph goes to the next page or not.
+ * to decide if the whole paragraph goes to the next page or not.
*/
sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin &rLine ) const
@@ -167,9 +167,9 @@ sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin &rLine ) const
// grow the requested area.
nHeight += pFrm->GrowTst( LONG_MAX );
- // The Grow() returnes the height, that the Upper of the TxtFrm
- // would grow the TxtFrm.
- // The TxtFrm itself can grow as much as it want.
+ // The Grow() returns the height by which the Upper of the TxtFrm
+ // would let the TxtFrm grow.
+ // The TxtFrm itself can grow as much as it wants.
bFit = nHeight >= nLineHeight;
}
}
@@ -188,7 +188,7 @@ sal_Bool SwTxtFrmBreak::IsBreakNow( SwTxtMargin &rLine )
SWAP_IF_SWAPPED( pFrm )
// bKeep is stronger than IsBreakNow()
- // Is enough space ?
+ // Is there enough space ?
if( bKeep || IsInside( rLine ) )
bBreak = sal_False;
else
@@ -255,13 +255,13 @@ WidowsAndOrphans::WidowsAndOrphans( SwTxtFrm *pNewFrm, const SwTwips nRst,
if( bKeep )
{
- // 5652: If pararagraph should not be splited but is larger than
+ // 5652: If pararagraph should not be split but is larger than
// the page, then bKeep is overruled.
if( bChkKeep && !pFrm->GetPrev() && !pFrm->IsInFtn() &&
pFrm->IsMoveable() &&
( !pFrm->IsInSct() || pFrm->FindSctFrm()->MoveAllowed(pFrm) ) )
bKeep = sal_False;
- // Even if Keep is set, the Orphans has to be regarded
+ // Even if Keep is set, Orphans has to be respected.
// e.g. if there are chained frames where a Follow in the last frame
// receives a Keep, because it is not (forward) movable -
// nevertheless the paragraph can request lines from the Master
@@ -360,8 +360,8 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, SwTxtMargin &rLine,
else
break;
}
- // Normaly Orphans are not taken into account for HasToFit.
- // But if Dummy-Lines are concerned and they violate the orphans rules
+ // Usually Orphans are not taken into account for HasToFit.
+ // But if Dummy-Lines are concerned and the Orphans rule is violated
// we make an exception: We leave behind one Dummyline and take
// the whole text to the next page/column.
if( rLine.GetLineNr() <= nOldOrphans &&
@@ -383,10 +383,10 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, SwTxtMargin &rLine,
* WidowsAndOrphans::FindWidows()
*************************************************************************/
-/* FindWidows positiones the SwTxtMargin of the Master to the line where to
+/* FindWidows positions the SwTxtMargin of the Master to the line where to
* break by examining and formatting the Follow.
* Returns sal_True if the Widows-rule matches, that means that the
- * paragraph should be not splitted (keep) !
+ * paragraph should not be split (keep) !
*/
sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
@@ -433,13 +433,13 @@ sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
// 8575: Follow to Master I
// If the Follow *grows*, there is the chance for the Master to
// receive lines, that it was forced to hand over to the Follow lately:
- // Prepare(Need); This query below of nChg!
+ // Prepare(Need); check that below nChg!
// (0W, 2O, 2M, 2F) + 1F = 3M, 2F
if( rLine.GetLineNr() > nWidLines && pFrame->IsJustWidow() )
{
- // Wenn der Master gelockt ist, so hat er vermutlich gerade erst
- // eine Zeile an uns abgegeben, diese geben nicht zurueck, nur
- // weil bei uns daraus mehrere geworden sind (z.B. durch Rahmen).
+ // If the Master is locked, it has probably just donated a line
+ // to us, we don't return that just because we turned it into
+ // multiple lines (e.g. via frames).
if( !pMaster->IsLocked() && pMaster->GetUpper() )
{
const SwTwips nTmpRstHeight = (pMaster->Frm().*fnRect->fnBottomDist)
@@ -549,7 +549,7 @@ sal_Bool WidowsAndOrphans::WouldFit( SwTxtMargin &rLine, SwTwips &rMaxHeight, sa
// Check the Widows-rule
if( !nWidLines && !pFrm->IsFollow() )
{
- // I.A. We only have to check for Widows if we are a Follow.
+ // Usually we only have to check for Widows if we are a Follow.
// On WouldFit the rule has to be checked for the Master too,
// because we are just in the middle of calculating the break.
// In Ctor of WidowsAndOrphans the nWidLines are only calced for
@@ -558,7 +558,7 @@ sal_Bool WidowsAndOrphans::WouldFit( SwTxtMargin &rLine, SwTwips &rMaxHeight, sa
nWidLines = rSet.GetWidows().GetValue();
}
- // Remain after Orphans/Initials enough lines for Widows?
+ // After Orphans/Initials, do enough lines remain for Widows?
// #111937#: If we are currently doing a test formatting, we may not
// consider the widows rule for two reasons:
// 1. The columns may have different widths.
commit 0121e7a8066e7f99723846011bbd4367cb6d993d
Author: gerhard oettl <lodev at ogersoft.at>
Date: Thu Sep 20 16:34:47 2012 +0200
fdo#39468: Translated German comments in sw/source/core/text
Change-Id: I53a415e1ec573d342e06d0cf10bad6f8cb95b478
diff --git a/sw/source/core/text/txttab.cxx b/sw/source/core/text/txttab.cxx
index a53cce9..0bd8b9a 100644
--- a/sw/source/core/text/txttab.cxx
+++ b/sw/source/core/text/txttab.cxx
@@ -314,9 +314,9 @@ SwTabPortion *SwTxtFormatter::NewTabPortion( SwTxtFormatInfo &rInf, bool bAuto )
}
}
- // Vorhandensein von Tabulatoren anzeigen ... ist nicht mehr noetig
+ // Show existence of tabs "..." is not necessary anymore
// pCurr->SetTabulation();
- // Aus Sicherheitsgruenden lassen wir uns die Daten errechnen
+ // We calculate the data as a precaution
// pTabPor->Height( pLast->Height() );
// pTabPor->SetAscent( pLast->GetAscent() );
return pTabPor;
@@ -375,13 +375,13 @@ sal_Bool SwTabPortion::PreFormat( SwTxtFormatInfo &rInf )
{
OSL_ENSURE( rInf.X() <= GetTabPos(), "SwTabPortion::PreFormat: rush hour" );
- // Hier lassen wir uns nieder...
+ // Here we settle down ...
Fix( static_cast<sal_uInt16>(rInf.X()) );
const bool bTabCompat = rInf.GetTxtFrm()->GetTxtNode()->getIDocumentSettingAccess()->get(IDocumentSettingAccess::TAB_COMPAT);
const bool bTabOverflow = rInf.GetTxtFrm()->GetTxtNode()->getIDocumentSettingAccess()->get(IDocumentSettingAccess::TAB_OVERFLOW);
- // Die Mindestbreite eines Tabs ist immer mindestens ein Blank
+ // The minimal width of a tab is one blank at least.
// #i37686# In compatibility mode, the minimum width
// should be 1, even for non-left tab stops.
sal_uInt16 nMinimumTabWidth = 1;
@@ -457,8 +457,7 @@ sal_Bool SwTabPortion::PreFormat( SwTxtFormatInfo &rInf )
if( bFull )
{
- // Wir muessen aufpassen, dass wir nicht endlos schleifen,
- // wenn die Breite kleiner ist, als ein Blank ...
+ // We have to look for endless loops, if the width is smaller than one blank
if( rInf.GetIdx() == rInf.GetLineStart() &&
// #119175# TabStop should be forced to current
// line if there is a fly reducing the line width:
@@ -479,8 +478,8 @@ sal_Bool SwTabPortion::PreFormat( SwTxtFormatInfo &rInf )
}
else
{
- // Ein Kunstgriff mit Effekt: Die neuen Tabportions verhalten sich nun
- // so, wie FlyFrms, die in der Zeile stehen - inklusive Adjustment !
+ // A trick with impact: The new Tabportions now behave like
+ // FlyFrms, located in the line - including adjustment !
SetFixWidth( PrtWidth() );
return sal_False;
}
@@ -527,8 +526,8 @@ sal_Bool SwTabPortion::PostFormat( SwTxtFormatInfo &rInf )
if( POR_TABCENTER == nWhich )
{
- // zentrierte Tabs bereiten Probleme:
- // Wir muessen den Anteil herausfinden, der noch auf die Zeile passt.
+ // centered tabs are problematic:
+ // We have to detect how much fits into the line.
KSHORT nNewWidth = nPorWidth /2;
if( nNewWidth > rInf.Width() - nRight )
nNewWidth = nPorWidth - (rInf.Width() - nRight);
@@ -543,14 +542,13 @@ sal_Bool SwTabPortion::PostFormat( SwTxtFormatInfo &rInf )
const KSHORT nAdjDiff = nDiffWidth - nPorWidth;
if( nAdjDiff > GetFixWidth() )
PrtWidth( nAdjDiff );
- // Nicht erschrecken: wir muessen rInf weiterschieben.
- // Immerhin waren wir als Rechtstab bislang nur ein Blank breit.
- // Da wir uns jetzt aufgespannt haben, muss der Differenzbetrag
- // auf rInf.X() addiert werden !
+ // Dont be afraid: we have to move rInf further.
+ // The right-tab till now only had the width of one blank.
+ // Now that we stretched, the difference had to be added to rInf.X() !
rInf.X( rInf.X() + PrtWidth() - nOldWidth );
}
SetFixWidth( PrtWidth() );
- // letzte Werte zuruecksetzen
+ // reset last values
rInf.SetLastTab(0);
if( POR_TABDECIMAL == nWhich )
rInf.SetTabDecimal(0);
@@ -567,7 +565,7 @@ sal_Bool SwTabPortion::PostFormat( SwTxtFormatInfo &rInf )
void SwTabPortion::Paint( const SwTxtPaintInfo &rInf ) const
{
#if OSL_DEBUG_LEVEL > 1
- // Wir wollen uns die Fixbreite anzeigen
+ // We want to view the fixed width
if( rInf.OnWin() && OPTDBG( rInf ) &&
!rInf.GetOpt().IsPagePreview() && \
!rInf.GetOpt().IsReadonly() && \
@@ -605,45 +603,45 @@ void SwTabPortion::Paint( const SwTxtPaintInfo &rInf ) const
if( rInf.OnWin() && pPortion && !pPortion->Width() )
pPortion->PrePaint( rInf, this );
- // Darstellung von Sonderzeichen
+ // display special characters
if( rInf.OnWin() && rInf.GetOpt().IsTab() )
{
- // gefuellte Tabs werden grau hinterlegt.
+ // filled tabs are shaded in gray
if( IsFilled() )
rInf.DrawViewOpt( *this, POR_TAB );
else
rInf.DrawTab( *this );
}
- // 6842: Tabs sollen auf einmal wieder unterstrichen werden.
+ // 6842: Tabs should be underlined at once.
if( rInf.GetFont()->IsPaintBlank() )
{
- // Tabs mit Fuellung
+ // tabs with filling / filled tabs
UniString aTxt = rtl::OUString(' ');
const KSHORT nCharWidth = rInf.GetTxtSize( aTxt ).Width();
// robust:
if( nCharWidth )
{
- // 6864: immer mit Kerning, auch auf dem Drucker!
+ // 6864: always with kerning, also on printer!
KSHORT nChar = Width() / nCharWidth;
rInf.DrawText( aTxt.Fill( nChar, ' ' ), *this, 0, nChar, sal_True );
}
}
- // Ausgabe von Fuellzeichen
+ // Display fill characters
if( IsFilled() )
{
- // Tabs mit Fuellung
+ // tabs with filling / filled tabs
UniString aTxt = rtl::OUString(cFill);
const KSHORT nCharWidth = rInf.GetTxtSize( aTxt ).Width();
OSL_ENSURE( nCharWidth, "!SwTabPortion::Paint: sophisticated tabchar" );
// robust:
if( nCharWidth )
{
- // 6864: immer mit Kerning, auch auf dem Drucker!
+ // 6864: always with kerning, also on printer!
KSHORT nChar = Width() / nCharWidth;
if ( cFill == '_' )
- ++nChar; // damit keine Luecken entstehen (Bug 13430)
+ ++nChar; // to avoid gaps (Bug 13430)
rInf.DrawText( aTxt.Fill( nChar, cFill ), *this, 0, nChar, sal_True );
}
}
diff --git a/sw/source/core/text/widorp.cxx b/sw/source/core/text/widorp.cxx
index 8f6249d..b761ce3 100644
--- a/sw/source/core/text/widorp.cxx
+++ b/sw/source/core/text/widorp.cxx
@@ -53,7 +53,7 @@
/*************************************************************************
* inline IsNastyFollow()
*************************************************************************/
-// Ein Follow, der auf der selben Seite steht, wie sein Master ist nasty.
+// A Follow on the same page as its master ist nasty.
inline sal_Bool IsNastyFollow( const SwTxtFrm *pFrm )
{
OSL_ENSURE( !pFrm->IsFollow() || !pFrm->GetPrev() ||
@@ -93,14 +93,13 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const SwTwips nRst )
}
/* BP 18.6.93: Widows.
- * Im Gegensatz zur ersten Implementierung werden die Widows nicht
- * mehr vorausschauend berechnet, sondern erst beim Formatieren des
- * gesplitteten Follows festgestellt. Im Master faellt die Widows-
- * Berechnung also generell weg (nWidows wird manipuliert).
- * Wenn der Follow feststellt, dass die Widowsregel zutrifft,
- * verschickt er an seinen Vorgaenger ein Prepare.
- * Ein besonderes Problem ergibt sich, wenn die Widows zuschlagen,
- * aber im Master noch ein paar Zeilen zur Verfuegung stehen.
+ * In contrast to the first implementation the Widows are not calculated
+ * in advance but detected when formating the splitted Follow.
+ * In Master the Widows-calculation is dropped completely
+ * (nWidows is manipulated). If the Follow detects that the
+ * Widows rule applies it sends a Prepare to its ancestor.
+ * A special problem is when the Widow rule applies but in Master
+ * there are some lines available.
*
*/
@@ -108,16 +107,15 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const SwTwips nRst )
* SwTxtFrmBreak::IsInside()
*************************************************************************/
-/* BP(22.07.92): Berechnung von Witwen und Waisen.
- * Die Methode liefert sal_True zurueck, wenn eine dieser Regelung zutrifft.
+/* BP(22.07.92): Calculation of Widows and Orphans.
+ * The method returns sal_True if one of the rules matches.
*
- * Eine Schwierigkeit gibt es im Zusammenhang mit Widows und
- * unterschiedlichen Formaten zwischen Master- und Folgeframes:
- * Beispiel: Wenn die erste Spalte 3cm und die zweite 4cm breit ist
- * und Widows auf sagen wir 3 gesetzt ist, so ist erst bei der Formatierung
- * des Follows entscheidbar, ob die Widowsbedingung einhaltbar ist oder
- * nicht. Leider ist davon abhaengig, ob der Absatz als Ganzes auf die
- * naechste Seite rutscht.
+ * One difficulty with Widows and different formats between
+ * Master- and Follow-Frame:
+ * Example: If the first column is 3cm and the second is 4cm and
+ * Widows is set to 3. The decision if the Widows rule matches can not
+ * be done until the Follow is formated. Unfortunately this is crucial
+ * sto decide if the whole paragraph goes to the next page or not.
*/
sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin &rLine ) const
@@ -136,18 +134,17 @@ sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin &rLine ) const
SwTwips nLineHeight = (*fnRect->fnYDiff)( nTmpY , nOrigin );
- // 7455 und 6114: Raum fuer die Umrandung unten einkalkulieren.
+ // 7455 und 6114: Calculate extra space for bottom border.
nLineHeight += (pFrm->*fnRect->fnGetBottomMargin)();
if( nRstHeight )
bFit = nRstHeight >= nLineHeight;
else
{
- // Der Frm besitzt eine Hoehe, mit der er auf die Seite passt.
+ // The Frm has a height to fit on the page.
SwTwips nHeight =
(*fnRect->fnYDiff)( (pFrm->GetUpper()->*fnRect->fnGetPrtBottom)(), nOrigin );
- // Wenn sich alles innerhalb des bestehenden Frames abspielt,
- // ist das Ergebnis sal_True;
+ // If everything is inside the existing frame the result is sal_True;
bFit = nHeight >= nLineHeight;
// --> OD #i103292#
@@ -165,14 +162,14 @@ sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin &rLine ) const
// <--
if( !bFit )
{
- // Die LineHeight sprengt die aktuelle Frm-Hoehe.
- // Nun rufen wir ein Probe-Grow, um zu ermitteln, ob der
- // Frame um den gewuenschten Bereich wachsen wuerde.
+ // The LineHeight exceeds the current Frm height.
+ // Call a test Grow to detect if the Frame could
+ // grow the requested area.
nHeight += pFrm->GrowTst( LONG_MAX );
- // Das Grow() returnt die Hoehe, um die der Upper des TxtFrm
- // den TxtFrm wachsen lassen wuerde.
- // Der TxtFrm selbst darf wachsen wie er will.
+ // The Grow() returnes the height, that the Upper of the TxtFrm
+ // would grow the TxtFrm.
+ // The TxtFrm itself can grow as much as it want.
bFit = nHeight >= nLineHeight;
}
}
@@ -190,20 +187,20 @@ sal_Bool SwTxtFrmBreak::IsBreakNow( SwTxtMargin &rLine )
{
SWAP_IF_SWAPPED( pFrm )
- // bKeep ist staerker als IsBreakNow()
- // Ist noch genug Platz ?
+ // bKeep is stronger than IsBreakNow()
+ // Is enough space ?
if( bKeep || IsInside( rLine ) )
bBreak = sal_False;
else
{
- /* Diese Klasse geht davon aus, dass der SwTxtMargin von Top nach Bottom
- * durchgearbeitet wird. Aus Performancegruenden wird in folgenden
- * Faellen der Laden fuer das weitere Aufspalten dicht gemacht:
- * Wenn eine einzige Zeile nicht mehr passt.
- * Sonderfall: bei DummyPortions ist LineNr == 1, obwohl wir splitten
- * wollen.
+ /* This class assumes that the SwTxtMargin is processed from Top to
+ * Bottom. Because of performance reasons we stop splitting in the
+ * following cases:
+ * If only one line does not fit.
+ * Special case: with DummyPortions there is LineNr == 1, though we
+ * want to split.
*/
- // 6010: DropLines mit einbeziehen
+ // 6010: include DropLines
sal_Bool bFirstLine = 1 == rLine.GetLineNr() && !rLine.GetPrev();
bBreak = sal_True;
@@ -258,16 +255,17 @@ WidowsAndOrphans::WidowsAndOrphans( SwTxtFrm *pNewFrm, const SwTwips nRst,
if( bKeep )
{
- // 5652: bei Absaetzen, die zusammengehalten werden sollen und
- // groesser sind als die Seite wird bKeep aufgehoben.
+ // 5652: If pararagraph should not be splited but is larger than
+ // the page, then bKeep is overruled.
if( bChkKeep && !pFrm->GetPrev() && !pFrm->IsInFtn() &&
pFrm->IsMoveable() &&
( !pFrm->IsInSct() || pFrm->FindSctFrm()->MoveAllowed(pFrm) ) )
bKeep = sal_False;
- //Auch bei gesetztem Keep muessen Orphans beachtet werden,
- //z.B. bei verketteten Rahmen erhaelt ein Follow im letzten Rahmen ein Keep,
- //da er nicht (vorwaerts) Moveable ist,
- //er darf aber trotzdem vom Master Zeilen anfordern wg. der Orphanregel.
+ // Even if Keep is set, the Orphans has to be regarded
+ // e.g. if there are chained frames where a Follow in the last frame
+ // receives a Keep, because it is not (forward) movable -
+ // nevertheless the paragraph can request lines from the Master
+ // because of the Orphan rule.
if( pFrm->IsFollow() )
nWidLines = pFrm->GetTxtNode()->GetSwAttrSet().GetWidows().GetValue();
}
@@ -302,8 +300,8 @@ WidowsAndOrphans::WidowsAndOrphans( SwTxtFrm *pNewFrm, const SwTwips nRst,
if( pFrm->IsInFtn() && !pFrm->GetIndPrev() )
{
- // Innerhalb von Fussnoten gibt es gute Gruende, das Keep-Attribut und
- // die Widows/Orphans abzuschalten.
+ // Inside of footnotes there are good reasons to turn off the Keep attribute
+ // as well as Widows/Orphans.
SwFtnFrm *pFtn = pFrm->FindFtnFrm();
sal_Bool bFt = !pFtn->GetAttr()->GetFtn().IsEndNote();
if( !pFtn->GetPrev() &&
@@ -329,8 +327,8 @@ WidowsAndOrphans::WidowsAndOrphans( SwTxtFrm *pNewFrm, const SwTwips nRst,
* WidowsAndOrphans::FindBreak()
*************************************************************************/
-/* Die Find*-Methoden suchen nicht nur, sondern stellen den SwTxtMargin auf
- * die Zeile ein, wo der Absatz gebrochen werden soll und kuerzen ihn dort.
+/* The Find*-Methodes do not only search, but adjust the SwTxtMargin to the
+ * line where the paragraph should have a break and truncate the paragraph there.
* FindBreak()
*/
@@ -362,11 +360,10 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, SwTxtMargin &rLine,
else
break;
}
- // Eigentlich werden bei HasToFit Schusterjungen (Orphans) nicht
- // beruecksichtigt, wenn allerdings Dummy-Lines im Spiel sind und
- // die Orphansregel verletzt wird, machen wir mal eine Ausnahme:
- // Wir lassen einfach eine Dummyline zurueck und wandern mit dem Text
- // komplett auf die naechste Seite/Spalte.
+ // Normaly Orphans are not taken into account for HasToFit.
+ // But if Dummy-Lines are concerned and they violate the orphans rules
+ // we make an exception: We leave behind one Dummyline and take
+ // the whole text to the next page/column.
if( rLine.GetLineNr() <= nOldOrphans &&
rLine.GetInfo().GetParaPortion()->IsDummy() &&
( ( bHasToFit && bRet ) || IsBreakNow( rLine ) ) )
@@ -386,10 +383,10 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, SwTxtMargin &rLine,
* WidowsAndOrphans::FindWidows()
*************************************************************************/
-/* FindWidows positioniert den SwTxtMargin des Masters auf die umzubrechende
- * Zeile, indem der Follow formatiert und untersucht wird.
- * Liefert sal_True zurueck, wenn die Widows-Regelung in Kraft tritt,
- * d.h. der Absatz _zusammengehalten_ werden soll !
+/* FindWidows positiones the SwTxtMargin of the Master to the line where to
+ * break by examining and formatting the Follow.
+ * Returns sal_True if the Widows-rule matches, that means that the
+ * paragraph should be not splitted (keep) !
*/
sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
@@ -408,12 +405,12 @@ sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
if( !pMaster )
return sal_False;
- // 5156: Wenn die erste Zeile des Follows nicht passt, wird der Master
- // wohl voll mit Dummies sein. In diesem Fall waere ein PREP_WIDOWS fatal.
+ // 5156: If the first line of the Follow does not fit, the master
+ // probably is full of Dummies. In this case a PREP_WIDOWS would be fatal.
if( pMaster->GetOfst() == pFrame->GetOfst() )
return sal_False;
- // Resthoehe des Masters
+ // Remaining height of the master
SWRECTFN( pFrame )
const SwTwips nDocPrtTop = (pFrame->*fnRect->fnGetPrtTop)();
@@ -430,13 +427,13 @@ sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
const SwTwips nChg = (*fnRect->fnYDiff)( nTmpY, nDocPrtTop + nOldHeight );
- // Unterhalb der Widows-Schwelle...
+ // below the Widows-treshold...
if( rLine.GetLineNr() >= nWidLines )
{
// 8575: Follow to Master I
- // Wenn der Follow *waechst*, so besteht fuer den Master die Chance,
- // Zeilen entgegenzunehmen, die er vor Kurzem gezwungen war an den
- // Follow abzugeben: Prepare(Need); diese Abfrage unterhalb von nChg!
+ // If the Follow *grows*, there is the chance for the Master to
+ // receive lines, that it was forced to hand over to the Follow lately:
+ // Prepare(Need); This query below of nChg!
// (0W, 2O, 2M, 2F) + 1F = 3M, 2F
if( rLine.GetLineNr() > nWidLines && pFrame->IsJustWidow() )
{
@@ -462,8 +459,7 @@ sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin &rLine )
}
// 8575: Follow to Master II
- // Wenn der Follow *schrumpft*, so besteht fuer den Master die Chance,
- // den kompletten Orphan zu inhalieren.
+ // If the Follow *shrinks*, maybe the Master can absorb the whole Orphan.
// (0W, 2O, 2M, 1F) - 1F = 3M, 0F -> PREP_ADJUST_FRM
// (0W, 2O, 3M, 2F) - 1F = 2M, 2F -> PREP_WIDOWS
@@ -527,11 +523,11 @@ sal_Bool WidowsAndOrphans::WouldFit( SwTxtMargin &rLine, SwTwips &rMaxHeight, sa
// Here it does not matter, if pFrm is swapped or not.
// IsInside() takes care for itself
- // Wir erwarten, dass rLine auf der letzten Zeile steht!!
+ // We expect that rLine is set to the last line
OSL_ENSURE( !rLine.GetNext(), "WouldFit: aLine::Bottom missed!" );
MSHORT nLineCnt = rLine.GetLineNr();
- // Erstmal die Orphansregel und den Initialenwunsch erfuellen ...
+ // First satisfy the Orphans-rule and the wish for initials ...
const MSHORT nMinLines = Max( GetOrphansLines(), rLine.GetDropLines() );
if ( nLineCnt < nMinLines )
return sal_False;
@@ -546,23 +542,23 @@ sal_Bool WidowsAndOrphans::WouldFit( SwTxtMargin &rLine, SwTwips &rMaxHeight, sa
nLineSum += rLine.GetLineHeight();
}
- // Wenn wir jetzt schon nicht mehr passen ...
+ // We do not fit
if( !IsInside( rLine ) )
return sal_False;
- // Jetzt noch die Widows-Regel ueberpruefen
+ // Check the Widows-rule
if( !nWidLines && !pFrm->IsFollow() )
{
- // I.A. brauchen Widows nur ueberprueft werden, wenn wir ein Follow
- // sind. Bei WouldFit muss aber auch fuer den Master die Regel ueber-
- // prueft werden, weil wir ja gerade erst die Trennstelle ermitteln.
- // Im Ctor von WidowsAndOrphans wurde nWidLines aber nur fuer Follows
- // aus dem AttrSet ermittelt, deshalb holen wir es hier nach:
+ // I.A. We only have to check for Widows if we are a Follow.
+ // On WouldFit the rule has to be checked for the Master too,
+ // because we are just in the middle of calculating the break.
+ // In Ctor of WidowsAndOrphans the nWidLines are only calced for
+ // Follows from the AttrSet - so we catch up now:
const SwAttrSet& rSet = pFrm->GetTxtNode()->GetSwAttrSet();
nWidLines = rSet.GetWidows().GetValue();
}
- // Sind nach Orphans/Initialen noch genug Zeilen fuer die Widows uebrig?
+ // Remain after Orphans/Initials enough lines for Widows?
// #111937#: If we are currently doing a test formatting, we may not
// consider the widows rule for two reasons:
// 1. The columns may have different widths.
More information about the Libreoffice-commits
mailing list