[PATCH] fdo#39468 Translate German Comments - rest of vcl/source/edi...

Christian M. Heller (via Code Review) gerrit at gerrit.libreoffice.org
Sat May 4 13:36:14 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3787

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/87/3787/1

fdo#39468 Translate German Comments - rest of vcl/source/edit/...

Change-Id: I6cb657853cf3e8d4e35c0dc09be6a4a15ba3a336
---
M vcl/source/edit/textund2.hxx
M vcl/source/edit/textundo.cxx
M vcl/source/edit/textview.cxx
M vcl/source/edit/vclmedit.cxx
M vcl/source/edit/xtextedt.cxx
5 files changed, 108 insertions(+), 170 deletions(-)



diff --git a/vcl/source/edit/textund2.hxx b/vcl/source/edit/textund2.hxx
index 17a2918..396317e 100644
--- a/vcl/source/edit/textund2.hxx
+++ b/vcl/source/edit/textund2.hxx
@@ -26,9 +26,9 @@
 class TextUndoDelPara : public TextUndo
 {
 private:
-    sal_Bool            mbDelObject;
-    sal_uLong           mnPara;
-    TextNode*       mpNode; // Zeigt auf das gueltige, nicht zerstoerte Objekt!
+    sal_Bool        mbDelObject;
+    sal_uLong       mnPara;
+    TextNode*       mpNode; // points at the valid not-destroyed object
 
 public:
                     TYPEINFO();
diff --git a/vcl/source/edit/textundo.cxx b/vcl/source/edit/textundo.cxx
index 3cbb85d..c37bfea 100644
--- a/vcl/source/edit/textundo.cxx
+++ b/vcl/source/edit/textundo.cxx
@@ -65,10 +65,6 @@
 
 } // namespace
 
-//
-// TextUndoManager
-// ===============
-//
 
 TextUndoManager::TextUndoManager( TextEngine* p )
 {
@@ -115,9 +111,6 @@
 void TextUndoManager::UndoRedoStart()
 {
     DBG_ASSERT( GetView(), "Undo/Redo: Active View?" );
-
-//  if ( GetView() )
-//      GetView()->HideSelection();
 }
 
 void TextUndoManager::UndoRedoEnd()
@@ -133,12 +126,6 @@
 
     mpTextEngine->FormatAndUpdate( GetView() );
 }
-
-
-//
-// TextUndo
-// ========
-//
 
 TextUndo::TextUndo( TextEngine* p )
 {
@@ -160,12 +147,6 @@
         GetView()->ImpSetSelection( rSel );
 }
 
-
-//
-// TextUndoDelPara
-// ===============
-//
-
 TextUndoDelPara::TextUndoDelPara( TextEngine* pTextEngine, TextNode* pNode, sal_uLong nPara )
                     : TextUndo( pTextEngine )
 {
@@ -183,7 +164,7 @@
 void TextUndoDelPara::Undo()
 {
     GetTextEngine()->InsertContent( mpNode, mnPara );
-    mbDelObject = sal_False;    // gehoert wieder der Engine
+    mbDelObject = sal_False;    // belongs again to the engine
 
     if ( GetView() )
     {
@@ -194,18 +175,17 @@
 
 void TextUndoDelPara::Redo()
 {
-    // pNode stimmt nicht mehr, falls zwischendurch Undos, in denen
-    // Absaetze verschmolzen sind.
+    // pNode is not valid anymore in case an Undo joined paragraphs
     mpNode = GetDoc()->GetNodes().GetObject( mnPara );
 
     delete GetTEParaPortions()->GetObject( mnPara );
     GetTEParaPortions()->Remove( mnPara );
 
-    // Node nicht loeschen, haengt im Undo!
+    // do not delte Node because of Undo!
     GetDoc()->GetNodes().Remove( mnPara );
     GetTextEngine()->ImpParagraphRemoved( mnPara );
 
-    mbDelObject = sal_True; // gehoert wieder dem Undo
+    mbDelObject = sal_True; // belongs again to the Undo
 
     sal_uLong nParas = GetDoc()->GetNodes().Count();
     sal_uLong n = mnPara < nParas ? mnPara : (nParas-1);
@@ -218,12 +198,6 @@
 {
     return ResId(STR_TEXTUNDO_DELPARA, *ImplGetResMgr());
 }
-
-
-//
-// TextUndoConnectParas
-// ====================
-//
 
 TextUndoConnectParas::TextUndoConnectParas( TextEngine* pTextEngine, sal_uLong nPara, sal_uInt16 nPos )
                     :   TextUndo( pTextEngine )
@@ -253,12 +227,6 @@
     return ResId(STR_TEXTUNDO_CONNECTPARAS, *ImplGetResMgr());
 }
 
-
-//
-// TextUndoSplitPara
-// =================
-//
-
 TextUndoSplitPara::TextUndoSplitPara( TextEngine* pTextEngine, sal_uLong nPara, sal_uInt16 nPos )
                     : TextUndo( pTextEngine )
 {
@@ -286,12 +254,6 @@
 {
     return ResId(STR_TEXTUNDO_SPLITPARA, *ImplGetResMgr());
 }
-
-
-//
-// TextUndoInsertChars
-// ===================
-//
 
 TextUndoInsertChars::TextUndoInsertChars( TextEngine* pTextEngine, const TextPaM& rTextPaM, const XubString& rStr )
                     : TextUndo( pTextEngine ),
@@ -341,13 +303,6 @@
     Shorten(sText);
     return OUString(ResId(STR_TEXTUNDO_INSERTCHARS, *ImplGetResMgr())).replaceAll("$1", sText);
 }
-
-
-
-//
-// TextUndoRemoveChars
-// ===================
-//
 
 TextUndoRemoveChars::TextUndoRemoveChars( TextEngine* pTextEngine, const TextPaM& rTextPaM, const XubString& rStr )
                     : TextUndo( pTextEngine ),
diff --git a/vcl/source/edit/textview.cxx b/vcl/source/edit/textview.cxx
index 4483616..5b5adde 100644
--- a/vcl/source/edit/textview.cxx
+++ b/vcl/source/edit/textview.cxx
@@ -177,9 +177,6 @@
     bool                mbCursorAtEndOfLine;
 };
 
-// -------------------------------------------------------------------------
-// (+) class TextView
-// -------------------------------------------------------------------------
 TextView::TextView( TextEngine* pEng, Window* pWindow ) :
     mpImpl(new ImpTextView)
 {
@@ -254,12 +251,11 @@
 
 void TextView::SetSelection( const TextSelection& rTextSel, sal_Bool bGotoCursor )
 {
-    // Falls jemand gerade ein leeres Attribut hinterlassen hat,
-    // und dann der Outliner die Selektion manipulitert:
+    // if someone left an empty attribute and then the Outliner manipulated the selection
     if ( !mpImpl->maSelection.HasRange() )
         mpImpl->mpTextEngine->CursorMoved( mpImpl->maSelection.GetStart().GetPara() );
 
-    // Wenn nach einem KeyInput die Selection manipuliert wird:
+    // if the selection is manipulated after a KeyInput
     mpImpl->mpTextEngine->CheckIdleFormatter();
 
     HideSelection();
@@ -303,8 +299,8 @@
         pSelection = NULL;
     else
     {
-        // Richtige Hintergrundfarbe einstellen.
-        // Ich bekomme leider nicht mit, ob sich diese inzwischen geaendert hat.
+        // set correct background color;
+        // unfortunately we cannot detect if it has changed
         Font aFont = mpImpl->mpTextEngine->GetFont();
         Color aColor = pOut->GetBackground().GetColor();
         aColor.SetTransparency( 0 );
@@ -353,8 +349,8 @@
         }
         else
         {
-            // Das VirtDev kann bei einem Resize sehr gross werden =>
-            // irgendwann mal kleiner machen!
+            // the VirtDev can get very large on Resize =>
+            // shrink now and then
             if ( ( aOutSz.Height() > ( rRect.GetHeight() + 20 ) ) ||
                  ( aOutSz.Width() > ( rRect.GetWidth() + 20 ) ) )
             {
@@ -367,7 +363,7 @@
         }
         if ( !bVDevValid )
         {
-            ImpPaint( rRect, sal_False /* ohne VDev */ );
+            ImpPaint( rRect, sal_False /* without VDev */ );
             return;
         }
 
@@ -420,7 +416,7 @@
                 if ( nPara == nEndPara )
                     nEndLine = pTEParaPortion->GetLineNumber( aSel.GetEnd().GetIndex(), sal_True );
 
-                // ueber die Zeilen iterieren....
+                // iterate over all lines
                 for ( sal_uInt16 nLine = nStartLine; nLine <= nEndLine; nLine++ )
                 {
                     TextLine* pLine = pTEParaPortion->GetLines()[ nLine ];
@@ -431,7 +427,7 @@
                     if ( ( nPara == nEndPara ) && ( nLine == nEndLine ) )
                         nEndIndex = aSel.GetEnd().GetIndex();
 
-                    // Kann passieren, wenn am Anfang einer umgebrochenen Zeile.
+                    // possible if at the beginning of a wrapped line
                     if ( nEndIndex < nStartIndex )
                         nEndIndex = nStartIndex;
 
@@ -446,7 +442,7 @@
                     Point aBottomRight( aTmpRect.BottomRight() );
                     aBottomRight.X()--;
 
-                    // Nur Painten, wenn im sichtbaren Bereich...
+                    // only paint if in the visible region
                     if ( ( aTopLeft.X() < aBottomRight.X() ) && ( aBottomRight.Y() >= aVisArea.Top() ) )
                     {
                         Point aPnt1( GetWindowPos( aTopLeft ) );
@@ -540,12 +536,12 @@
     sal_Bool bDone      = sal_True;
     sal_Bool bModified  = sal_False;
     sal_Bool bMoved     = sal_False;
-    sal_Bool bEndKey    = sal_False;    // spezielle CursorPosition
+    sal_Bool bEndKey    = sal_False;    // special CursorPosition
     sal_Bool bAllowIdle = sal_True;
 
-    // Um zu pruefen ob durch irgendeine Aktion mModified, das lokale
-    // bModified wird z.B. bei Cut/Paste nicht gesetzt, weil dort an anderen
-    // Stellen das updaten erfolgt.
+    // check mModified;
+    // the local bModified is not set e.g. by Cut/Paste, as here
+    // the update happens somewhere else
     sal_Bool bWasModified = mpImpl->mpTextEngine->IsModified();
     mpImpl->mpTextEngine->SetModified( sal_False );
 
@@ -588,7 +584,7 @@
             }
             break;
 
-            default:    // wird dann evtl. unten bearbeitet.
+            default:    // might get processed below
                         eFunc = KEYFUNC_DONTKNOW;
         }
     }
@@ -717,8 +713,8 @@
             break;
             case KEY_RETURN:
             {
-                // Shift-RETURN darf nicht geschluckt werden, weil dann keine
-                // mehrzeilige Eingabe in Dialogen/Property-Editor moeglich.
+                // do not swallow Shift-RETURN, as this would disable multi-line entries
+                // in dialogs & property editors
                 if ( !mpImpl->mbReadOnly && !rKeyEvent.GetKeyCode().IsMod1() &&
                         !rKeyEvent.GetKeyCode().IsMod2() && ImplCheckTextLen( OUString('x') ) )
                 {
@@ -755,7 +751,7 @@
                 if ( TextEngine::IsSimpleCharInput( rKeyEvent ) )
                 {
                     sal_Unicode nCharCode = rKeyEvent.GetCharCode();
-                    if ( !mpImpl->mbReadOnly && ImplCheckTextLen( OUString(nCharCode) ) )    // sonst trotzdem das Zeichen schlucken...
+                    if ( !mpImpl->mbReadOnly && ImplCheckTextLen( OUString(nCharCode) ) )    // otherwise swallow the character anyway
                     {
                         aCurSel = mpImpl->mpTextEngine->ImpInsertText( nCharCode, aCurSel, !IsInsertMode(), sal_True );
                         bModified = sal_True;
@@ -777,7 +773,7 @@
 
     if ( bModified )
     {
-        // Idle-Formatter nur, wenn AnyInput.
+        // Idle-Formatter only if AnyInput
         if ( bAllowIdle && Application::AnyInput( VCL_INPUT_KEYBOARD) )
             mpImpl->mpTextEngine->IdleFormatAndUpdate( this );
         else
@@ -785,7 +781,7 @@
     }
     else if ( bMoved )
     {
-        // Selection wird jetzt gezielt in ImpMoveCursor gemalt.
+        // selection is painted now in ImpMoveCursor
         ImpShowCursor( mpImpl->mbAutoScroll, sal_True, bEndKey );
     }
 
@@ -819,7 +815,7 @@
 
 void TextView::MouseButtonDown( const MouseEvent& rMouseEvent )
 {
-    mpImpl->mpTextEngine->CheckIdleFormatter();    // Falls schnelles Tippen und MouseButtonDown
+    mpImpl->mpTextEngine->CheckIdleFormatter();    // for fast typing and MouseButtonDown
     mpImpl->mnTravelXPos = TRAVEL_X_DONTKNOW;
     mpImpl->mbClickedInSelection = IsSelectionAtPoint( rMouseEvent.GetPosPixel() );
 
@@ -833,19 +829,19 @@
     // we need to reevaluate the selection after the notification has been fired.
     mpImpl->mbClickedInSelection = IsSelectionAtPoint( rMouseEvent.GetPosPixel() );
 
-    // Sonderbehandlungen
+    // special cases
     if ( !rMouseEvent.IsShift() && ( rMouseEvent.GetClicks() >= 2 ) )
     {
         if ( rMouseEvent.IsMod2() )
         {
             HideSelection();
             ImpSetSelection( mpImpl->maSelection.GetEnd() );
-            SetCursorAtPoint( rMouseEvent.GetPosPixel() );  // Wird von SelectionEngine bei MOD2 nicht gesetzt
+            SetCursorAtPoint( rMouseEvent.GetPosPixel() );  // not set by SelectionEngine for MOD2
         }
 
         if ( rMouseEvent.GetClicks() == 2 )
         {
-            // Wort selektieren
+            // select word
             if ( mpImpl->maSelection.GetEnd().GetIndex() < mpImpl->mpTextEngine->GetTextLen( mpImpl->maSelection.GetEnd().GetPara() ) )
             {
                 HideSelection();
@@ -882,7 +878,7 @@
         }
         else if ( rMouseEvent.GetClicks() == 3 )
         {
-            // Absatz selektieren
+            // select paragraph
             if ( mpImpl->maSelection.GetStart().GetIndex() || ( mpImpl->maSelection.GetEnd().GetIndex() < mpImpl->mpTextEngine->GetTextLen( mpImpl->maSelection.GetEnd().GetPara() ) ) )
             {
                 HideSelection();
@@ -906,7 +902,7 @@
 
 void TextView::Command( const CommandEvent& rCEvt )
 {
-    mpImpl->mpTextEngine->CheckIdleFormatter();    // Falls schnelles Tippen und MouseButtonDown
+    mpImpl->mpTextEngine->CheckIdleFormatter();    // for fast typing and MouseButtonDown
     mpImpl->mpTextEngine->SetActiveView( this );
 
     if ( rCEvt.GetCommand() == COMMAND_STARTEXTTEXTINPUT )
@@ -919,7 +915,7 @@
     }
     else if ( rCEvt.GetCommand() == COMMAND_ENDEXTTEXTINPUT )
     {
-        DBG_ASSERT( mpImpl->mpTextEngine->mpIMEInfos, "COMMAND_ENDEXTTEXTINPUT => Kein Start ?" );
+        DBG_ASSERT( mpImpl->mpTextEngine->mpIMEInfos, "COMMAND_ENDEXTTEXTINPUT => No Start ?" );
         if( mpImpl->mpTextEngine->mpIMEInfos )
         {
             TEParaPortion* pPortion = mpImpl->mpTextEngine->mpTEParaPortions->GetObject( mpImpl->mpTextEngine->mpIMEInfos->aPos.GetPara() );
@@ -940,7 +936,7 @@
     }
     else if ( rCEvt.GetCommand() == COMMAND_EXTTEXTINPUT )
     {
-        DBG_ASSERT( mpImpl->mpTextEngine->mpIMEInfos, "COMMAND_EXTTEXTINPUT => Kein Start ?" );
+        DBG_ASSERT( mpImpl->mpTextEngine->mpIMEInfos, "COMMAND_EXTTEXTINPUT => No Start ?" );
         if( mpImpl->mpTextEngine->mpIMEInfos )
         {
             const CommandExtTextInputData* pData = rCEvt.GetExtTextInputData();
@@ -1044,7 +1040,7 @@
 
 void TextView::ShowCursor( sal_Bool bGotoCursor, sal_Bool bForceVisCursor )
 {
-    // Die Einstellung hat mehr Gewicht:
+    // this setting has more weight
     if ( !mpImpl->mbAutoScroll )
         bGotoCursor = sal_False;
     ImpShowCursor( bGotoCursor, bForceVisCursor, sal_False );
@@ -1057,7 +1053,7 @@
 
 void TextView::Scroll( long ndX, long ndY )
 {
-    DBG_ASSERT( mpImpl->mpTextEngine->IsFormatted(), "Scroll: Nicht formatiert!" );
+    DBG_ASSERT( mpImpl->mpTextEngine->IsFormatted(), "Scroll: Not formatted!" );
 
     if ( !ndX && !ndY )
         return;
@@ -1122,7 +1118,7 @@
     {
         TETextDataObject* pDataObj = new TETextDataObject( GetSelected() );
 
-        if ( mpImpl->mpTextEngine->HasAttrib( TEXTATTR_HYPERLINK ) )  // Dann auch als HTML
+        if ( mpImpl->mpTextEngine->HasAttrib( TEXTATTR_HYPERLINK ) )  // then also as HTML
             mpImpl->mpTextEngine->Write( pDataObj->GetHTMLStream(), &mpImpl->maSelection, sal_True );
 
         const sal_uInt32 nRef = Application::ReleaseSolarMutex();
@@ -1236,7 +1232,7 @@
 
 TextSelection TextView::ImpMoveCursor( const KeyEvent& rKeyEvent )
 {
-    // Eigentlich nur bei Up/Down noetig, aber was solls.
+    // normally only needed for Up/Down; but who cares
     mpImpl->mpTextEngine->CheckIdleFormatter();
 
     TextPaM aPaM( mpImpl->maSelection.GetEnd() );
@@ -1312,7 +1308,7 @@
                             break;
     }
 
-    // Bewirkt evtl. ein CreateAnchor oder Deselection all
+    // might cause a CreateAnchor or Deselection all
     mpImpl->mpSelEngine->CursorPosChanging( bSelect, aTranslatedKeyEvent.GetKeyCode().IsMod1() );
 
     if ( aOldEnd != aPaM )
@@ -1323,7 +1319,7 @@
         aNewSelection.GetEnd() = aPaM;
         if ( bSelect )
         {
-            // Dann wird die Selektion erweitert...
+            // extend the selection
             ImpSetSelection( aNewSelection );
             ShowSelection( TextSelection( aOldEnd, aPaM ) );
         }
@@ -1468,7 +1464,7 @@
 
 TextPaM TextView::ImpDelete( sal_uInt8 nMode, sal_uInt8 nDelMode )
 {
-    if ( mpImpl->maSelection.HasRange() )  // dann nur Sel. loeschen
+    if ( mpImpl->maSelection.HasRange() )  // only delete selection
         return mpImpl->mpTextEngine->ImpDeleteText( mpImpl->maSelection );
 
     TextPaM aStartPaM = mpImpl->maSelection.GetStart();
@@ -1495,7 +1491,7 @@
                 aEndPaM.GetIndex() = 0;
             else if ( aEndPaM.GetPara() )
             {
-                // Absatz davor
+                // previous paragraph
                 aEndPaM.GetPara()--;
                 aEndPaM.GetIndex() = 0;
             }
@@ -1521,7 +1517,7 @@
                 aEndPaM.GetIndex() = pNode->GetText().Len();
             else if ( aEndPaM.GetPara() < ( mpImpl->mpTextEngine->mpDoc->GetNodes().Count() - 1 ) )
             {
-                // Absatz danach
+                // next paragraph
                 aEndPaM.GetPara()++;
                 TextNode* pNextNode = mpImpl->mpTextEngine->mpDoc->GetNodes().GetObject( aEndPaM.GetPara() );
                 aEndPaM.GetIndex() = pNextNode->GetText().Len();
@@ -1549,18 +1545,18 @@
 
     TEParaPortion* pPPortion = mpImpl->mpTextEngine->mpTEParaPortions->GetObject( rPaM.GetPara() );
     sal_uInt16 nLine = pPPortion->GetLineNumber( rPaM.GetIndex(), sal_False );
-    if ( nLine )    // gleicher Absatz
+    if ( nLine )    // same paragraph
     {
         sal_uInt16 nCharPos = mpImpl->mpTextEngine->GetCharPos( rPaM.GetPara(), nLine-1, nX );
         aPaM.GetIndex() = nCharPos;
-        // Wenn davor eine autom.Umgebrochene Zeile, und ich muss genau an das
-        // Ende dieser Zeile, landet der Cursor in der aktuellen Zeile am Anfang
-        // Siehe Problem: Letztes Zeichen einer autom.umgebr. Zeile = Cursor
+        // If we need to go to the end of a line that was wrapped automatically,
+        // the cursor ends up at the beginning of the 2nd line
+        // Problem: Last character of an automatically wrapped line = Cursor
         TextLine* pLine = pPPortion->GetLines()[ nLine - 1 ];
         if ( aPaM.GetIndex() && ( aPaM.GetIndex() == pLine->GetEnd() ) )
             aPaM.GetIndex()--;
     }
-    else if ( rPaM.GetPara() )  // vorheriger Absatz
+    else if ( rPaM.GetPara() )  // previous paragraph
     {
         aPaM.GetPara()--;
         pPPortion = mpImpl->mpTextEngine->mpTEParaPortions->GetObject( aPaM.GetPara() );
@@ -1592,12 +1588,12 @@
         sal_uInt16 nCharPos = mpImpl->mpTextEngine->GetCharPos( rPaM.GetPara(), nLine+1, nX );
         aPaM.GetIndex() = nCharPos;
 
-        // Sonderbehandlung siehe CursorUp...
+        // special case CursorUp
         TextLine* pLine = pPPortion->GetLines()[ nLine + 1 ];
         if ( ( aPaM.GetIndex() == pLine->GetEnd() ) && ( aPaM.GetIndex() > pLine->GetStart() ) && aPaM.GetIndex() < pPPortion->GetNode()->GetText().Len() )
             aPaM.GetIndex()--;
     }
-    else if ( rPaM.GetPara() < ( mpImpl->mpTextEngine->mpDoc->GetNodes().Count() - 1 ) )   // naechster Absatz
+    else if ( rPaM.GetPara() < ( mpImpl->mpTextEngine->mpDoc->GetNodes().Count() - 1 ) )   // next paragraph
     {
         aPaM.GetPara()++;
         pPPortion = mpImpl->mpTextEngine->mpTEParaPortions->GetObject( aPaM.GetPara() );
@@ -1632,14 +1628,14 @@
     TextLine* pLine = pPPortion->GetLines()[ nLine ];
     aPaM.GetIndex() = pLine->GetEnd();
 
-    if ( pLine->GetEnd() > pLine->GetStart() )  // Leerzeile
+    if ( pLine->GetEnd() > pLine->GetStart() )  // empty line
     {
         sal_Unicode cLastChar = pPPortion->GetNode()->GetText().GetChar((sal_uInt16)(aPaM.GetIndex()-1) );
         if ( ( cLastChar == ' ' ) && ( aPaM.GetIndex() != pPPortion->GetNode()->GetText().Len() ) )
         {
-            // Bei einem Blank in einer autom. umgebrochenen Zeile macht es Sinn,
-            // davor zu stehen, da der Anwender hinter das Wort will.
-            // Wenn diese geaendert wird, Sonderbehandlung fuer Pos1 nach End!
+            // for a blank in an automatically-wrapped line it is better to stand before it,
+            // as the user will intend to stand behind the prior word.
+            // If there is a change, special case for Pos1 after End!
             aPaM.GetIndex()--;
         }
     }
@@ -1789,18 +1785,18 @@
         {
             aNewStartPos.X() += ( aEditCursor.Right() - nVisEndX );
 
-            // Darfs ein bischen mehr sein?
+            // do you want some more?
             aNewStartPos.X() += nMoreX;
         }
         else if ( aEditCursor.Left() <= nVisStartX )
         {
             aNewStartPos.X() -= ( nVisStartX - aEditCursor.Left() );
 
-            // Darfs ein bischen mehr sein?
+            // do you want some more?
             aNewStartPos.X() -= nMoreX;
         }
 
-        // X kann durch das 'bischen mehr' falsch sein:
+        // X can be wrong for the 'some more' above:
 //      sal_uLong nMaxTextWidth = mpImpl->mpTextEngine->GetMaxTextWidth();
 //      if ( !nMaxTextWidth || ( nMaxTextWidth > 0x7FFFFFFF ) )
 //          nMaxTextWidth = 0x7FFFFFFF;
@@ -1814,7 +1810,7 @@
         else if ( aNewStartPos.X() > nMaxX )
             aNewStartPos.X() = nMaxX;
 
-        // Y sollte nicht weiter unten als noetig liegen:
+        // Y should not be further down than needed
         long nYMax = mpImpl->mpTextEngine->GetTextHeight() - aOutSz.Height();
         if ( nYMax < 0 )
             nYMax = 0;
@@ -1847,7 +1843,7 @@
 
     TextPaM aPaM = mpImpl->mpTextEngine->GetPaM( aDocPos );
 
-    // aTmpNewSel: Diff zwischen alt und neu, nicht die neue Selektion
+    // aTmpNewSel: Diff between old and new; not the new selection
     TextSelection aTmpNewSel( mpImpl->maSelection.GetEnd(), aPaM );
     TextSelection aNewSel( mpImpl->maSelection );
     aNewSel.GetEnd() = aPaM;
@@ -1877,10 +1873,10 @@
 
     Point aDocPos = GetDocPos( rPosPixel );
     TextPaM aPaM = mpImpl->mpTextEngine->GetPaM( aDocPos, sal_False );
-    // Bei Hyperlinks D&D auch ohne Selektion starten.
-    // BeginDrag wird aber nur gerufen, wenn IsSelectionAtPoint()
-    // Problem: IsSelectionAtPoint wird bei Command() nicht gerufen,
-    // wenn vorher im MBDown schon sal_False returnt wurde.
+    // For Hyperlinks D&D also start w/o a selection.
+    // BeginDrag is only called, however, if IsSelectionAtPoint()
+    // Problem: IsSelectionAtPoint is not called by Command()
+    // if before MBDown returned sal_False.
     return ( IsInSelection( aPaM ) ||
             ( /* mpImpl->mpSelEngine->IsInCommand() && */ mpImpl->mpTextEngine->FindAttrib( aPaM, TEXTATTR_HYPERLINK ) ) );
 }
@@ -1993,7 +1989,7 @@
         n += rNewText.Len();
         if ( n > mpImpl->mpTextEngine->GetMaxTextLen() )
         {
-            // nur dann noch ermitteln, wie viel Text geloescht wird
+            // calculate how much text is being deleted
             n -= mpImpl->mpTextEngine->GetTextLen( mpImpl->maSelection );
             if ( n > mpImpl->mpTextEngine->GetMaxTextLen() )
                 bOK = sal_False;
@@ -2016,14 +2012,14 @@
 
         TETextDataObject* pDataObj = new TETextDataObject( GetSelected() );
 
-        if ( mpImpl->mpTextEngine->HasAttrib( TEXTATTR_HYPERLINK ) )  // Dann auch als HTML
+        if ( mpImpl->mpTextEngine->HasAttrib( TEXTATTR_HYPERLINK ) )  // then also as HTML
             mpImpl->mpTextEngine->Write( pDataObj->GetHTMLStream(), &mpImpl->maSelection, sal_True );
 
 
         /*
-        // D&D eines Hyperlinks.
-        // Besser waere es im MBDown sich den MBDownPaM zu merken,
-        // ist dann aber inkompatibel => spaeter mal umstellen.
+        // D&D of a Hyperlink
+        // TODO: Better would be to store MBDownPaM in MBDown,
+        // but this would be incompatible => change later
         TextPaM aPaM( mpImpl->mpTextEngine->GetPaM( GetDocPos( GetWindow()->GetPointerPosPixel() ) ) );
         const TextCharAttrib* pAttr = mpImpl->mpTextEngine->FindCharAttrib( aPaM, TEXTATTR_HYPERLINK );
         if ( pAttr )
@@ -2066,7 +2062,7 @@
     {
         ImpHideDDCursor();
 
-        // Daten fuer das loeschen nach einem DROP_MOVE:
+        // Data for deleting after DROP_MOVE:
         TextSelection aPrevSel( mpImpl->maSelection );
         aPrevSel.Justify();
         sal_uLong nPrevParaCount = mpImpl->mpTextEngine->GetParagraphCount();
@@ -2114,7 +2110,7 @@
                 !mpImpl->mbSupportProtectAttribute && // don't remove currently selected element
                 (( rDTDE.DropAction & datatransfer::dnd::DNDConstants::ACTION_MOVE ) || !bStarterOfDD) )
         {
-            // ggf. Selection anpasssen:
+            // adjust selection if necessary
             if ( ( mpImpl->mpDDInfo->maDropPos.GetPara() < aPrevSel.GetStart().GetPara() ) ||
                  ( ( mpImpl->mpDDInfo->maDropPos.GetPara() == aPrevSel.GetStart().GetPara() )
                         && ( mpImpl->mpDDInfo->maDropPos.GetIndex() < aPrevSel.GetStart().GetIndex() ) ) )
@@ -2139,7 +2135,7 @@
             }
             else
             {
-                // aktuelle Selektion anpassen
+                // adjust current selection
                 TextPaM aPaM = mpImpl->maSelection.GetStart();
                 aPaM.GetPara() -= ( aPrevSel.GetEnd().GetPara() - aPrevSel.GetStart().GetPara() );
                 if ( aPrevSel.GetEnd().GetPara() == mpImpl->mpDDInfo->maDropPos.GetPara() )
@@ -2208,7 +2204,7 @@
     }
     else
     {
-        // Alten Cursor wegzeichnen...
+        // delete old Cursor
         if ( !mpImpl->mpDDInfo->mbVisCursor || ( aPrevDropPos != mpImpl->mpDDInfo->maDropPos ) )
         {
             ImpHideDDCursor();
@@ -2231,7 +2227,7 @@
 
 Point TextView::GetDocPos( const Point& rWindowPos ) const
 {
-    // Fensterposition => Dokumentposition
+    // Window Position => Document Position
 
     Point aPoint;
 
@@ -2252,7 +2248,7 @@
 
 Point TextView::GetWindowPos( const Point& rDocPos ) const
 {
-    // Dokumentposition => Fensterposition
+    // Document Position => Window Position
 
     Point aPoint;
 
@@ -2305,7 +2301,7 @@
 //  aSel.GetStart() = aSel.GetEnd();
 //  mpView->SetSelection( aSel );
 
-    // Es darf kein ShowCursor folgen:
+    // may not be followed by ShowCursor
     mpView->HideSelection();
     mpView->ImpSetSelection( mpView->mpImpl->maSelection.GetEnd() );
 }
@@ -2327,12 +2323,12 @@
 
 void TextSelFunctionSet::DeselectAtPoint( const Point& )
 {
-    // Nur bei Mehrfachselektion
+    // only for multiple selection
 }
 
 void TextSelFunctionSet::DestroyAnchor()
 {
-    // Nur bei Mehrfachselektion
+    // only for multiple selection
 }
 TextEngine*         TextView::GetTextEngine() const
 { return mpImpl->mpTextEngine; }
diff --git a/vcl/source/edit/vclmedit.cxx b/vcl/source/edit/vclmedit.cxx
index f4993d0..e112fc7 100644
--- a/vcl/source/edit/vclmedit.cxx
+++ b/vcl/source/edit/vclmedit.cxx
@@ -340,13 +340,6 @@
     return 0;
 }
 
-
-// void ImpVclMEdit::ImpModified()
-// {
-//  // Wann wird das gerufen ?????????????????????
-//  pVclMultiLineEdit->Modify();
-// }
-
 void ImpVclMEdit::SetAlign( WinBits nWinStyle )
 {
     sal_Bool bRTL = Application::GetSettings().GetLayoutRTL();
@@ -373,7 +366,7 @@
 void ImpVclMEdit::SetReadOnly( sal_Bool bRdOnly )
 {
     mpTextWindow->GetTextView()->SetReadOnly( bRdOnly );
-    // Farbe anpassen ???????????????????????????
+    // TODO: Adjust color?
 }
 
 sal_Bool ImpVclMEdit::IsReadOnly() const
@@ -608,10 +601,10 @@
     maSelection = Selection();
     TextSelection aTextSel( mpTextWindow->GetTextView()->GetSelection() );
     aTextSel.Justify();
-    // Selektion flachklopfen => jeder Umbruch ein Zeichen...
+    // flatten selection => every line-break a character
 
     ExtTextEngine* pExtTextEngine = mpTextWindow->GetTextEngine();
-    // Absaetze davor:
+    // paragraphs before
     sal_uLong n;
     for ( n = 0; n < aTextSel.GetStart().GetPara(); n++ )
     {
@@ -619,7 +612,7 @@
         maSelection.Min()++;
     }
 
-    // Erster Absatz mit Selektion:
+    // first paragraph with selection
     maSelection.Max() = maSelection.Min();
     maSelection.Min() += aTextSel.GetStart().GetIndex();
 
@@ -749,7 +742,7 @@
     if ( !mbTextSelectable )
         return;
 
-    mbInMBDown = sal_True;  // Dann im GetFocus nicht alles selektieren wird
+    mbInMBDown = sal_True;  // so that GetFocus does not select everything
     mpExtTextView->MouseButtonDown( rMEvt );
     Window::MouseButtonDown( rMEvt );
     GrabFocus();
@@ -777,7 +770,7 @@
     {
         if ( Edit::GetGetSpecialCharsFunction() )
         {
-            // Damit die Selektion erhalten bleibt
+            // to maintain the selection
             mbActivePopup = sal_True;
             OUString aChars = Edit::GetGetSpecialCharsFunction()( this, GetFont() );
             if (!aChars.isEmpty())
@@ -849,7 +842,7 @@
         Point aPos = rCEvt.GetMousePosPixel();
         if ( !rCEvt.IsMouseEvent() )
         {
-            // !!! Irgendwann einmal Menu zentriert in der Selektion anzeigen !!!
+            // Sometime do show Menu centered in the selection !!!
             Size aSize = GetOutputSizePixel();
             aPos = Point( aSize.Width()/2, aSize.Height()/2 );
         }
@@ -909,7 +902,7 @@
                 && ( mbSelectOnTab &&
                     (!mbInMBDown || ( GetSettings().GetStyleSettings().GetSelectionOptions() & SELECTION_OPTION_FOCUS ) )) )
         {
-            // Alles selektieren, aber nicht scrollen
+            // select everything, but do not scroll
             sal_Bool bAutoScroll = mpExtTextView->IsAutoScroll();
             mpExtTextView->SetAutoScroll( sal_False );
             mpExtTextView->SetSelection( TextSelection( TextPaM( 0, 0 ), TextPaM( 0xFFFF, 0xFFFF ) ) );
@@ -999,8 +992,8 @@
 {
     const StyleSettings& rStyleSettings = GetSettings().GetStyleSettings();
 
-    // Der Font muss immer mit manipuliert werden, weil die TextEngine
-    // sich nicht um TextColor/Background kuemmert
+    // The Font has to be adjusted, as the TextEngine does not take care of
+    // TextColor/Background
 
     Color aTextColor = rStyleSettings.GetFieldTextColor();
     if ( IsControlForeground() )
@@ -1039,8 +1032,7 @@
                 pImpVclMEdit->GetTextWindow()->SetBackground( GetControlBackground() );
             else
                 pImpVclMEdit->GetTextWindow()->SetBackground( rStyleSettings.GetFieldColor() );
-            // Auch am VclMultiLineEdit einstellen, weil die TextComponent
-            // ggf. die Scrollbars hidet.
+            // also adjust for VclMultiLineEdit as the TextComponent might hide Scrollbars
             SetBackground( pImpVclMEdit->GetTextWindow()->GetBackground() );
         }
     }
@@ -1223,7 +1215,7 @@
     sal_Int32 nLeft, nTop, nRight, nBottom;
     ((Window*)this)->GetBorder( nLeft, nTop, nRight, nBottom );
 
-    // In der Hoehe auf ganze Zeilen justieren
+    // center vertically for whole lines
 
     long nHeight = aSz.Height() - nTop - nBottom;
     long nLineHeight = pImpVclMEdit->CalcSize( 1, 1 ).Height();
@@ -1354,7 +1346,7 @@
         }
     }
 
-    // Inhalt
+    // contents
     if ( ( nFlags & WINDOW_DRAW_MONO ) || ( eOutDevType == OUTDEV_PRINTER ) )
         pDev->SetTextColor( Color( COL_BLACK ) );
     else
@@ -1385,7 +1377,7 @@
     {
         Rectangle aClip( aPos, aSize );
         if ( aTextSz.Height() > aSize.Height() )
-            aClip.Bottom() += aTextSz.Height() - aSize.Height() + 1;  // Damit HP-Drucker nicht 'weg-optimieren'
+            aClip.Bottom() += aTextSz.Height() - aSize.Height() + 1;  // so that HP-printer does not 'optimize-away'
         pDev->IntersectClipRegion( aClip );
     }
 
@@ -1503,7 +1495,7 @@
 }
 
 //
-// Internas fuer abgeleitete Klassen, z.B. TextComponent
+// Internals for derived classes, e.g. TextComponent
 
 ExtTextEngine* VclMultiLineEdit::GetTextEngine() const
 {
diff --git a/vcl/source/edit/xtextedt.cxx b/vcl/source/edit/xtextedt.cxx
index e9b4e73..a22d88e 100644
--- a/vcl/source/edit/xtextedt.cxx
+++ b/vcl/source/edit/xtextedt.cxx
@@ -27,10 +27,6 @@
 using namespace ::com::sun::star;
 
 
-
-// -------------------------------------------------------------------------
-// class ExtTextEngine
-// -------------------------------------------------------------------------
 ExtTextEngine::ExtTextEngine() : maGroupChars(OUString("(){}[]"))
 {
 }
@@ -52,7 +48,7 @@
         {
             if ( ( nMatchChar % 2 ) == 0 )
             {
-                // Vorwaerts suchen...
+                // search forwards
                 sal_Unicode nSC = maGroupChars.GetChar( nMatchChar );
                 sal_Unicode nEC = maGroupChars.GetChar( nMatchChar+1 );
 
@@ -80,7 +76,7 @@
                         nCur = 0;
                     }
                 }
-                if ( nLevel == 0 )  // gefunden
+                if ( nLevel == 0 )  // found
                 {
                     aSel.GetStart() = rCursor;
                     aSel.GetEnd() = TextPaM( nPara, nCur+1 );
@@ -88,7 +84,7 @@
             }
             else
             {
-                // Rueckwaerts suchen...
+                // search backwards
                 sal_Unicode nEC = maGroupChars.GetChar( nMatchChar );
                 sal_Unicode nSC = maGroupChars.GetChar( nMatchChar-1 );
 
@@ -119,17 +115,17 @@
                         if ( nPara )
                         {
                             nPara--;
-                            nCur = GetTextLen( nPara )-1;   // egal ob negativ, weil if Len()
+                            nCur = GetTextLen( nPara )-1;   // no matter if negativ, as if Len()
                         }
                         else
                             break;
                     }
                 }
 
-                if ( nLevel == 0 )  // gefunden
+                if ( nLevel == 0 )  // found
                 {
                     aSel.GetStart() = rCursor;
-                    aSel.GetStart().GetIndex()++;   // hinter das Zeichen
+                    aSel.GetStart().GetIndex()++;   // behind the char
                     aSel.GetEnd() = TextPaM( nPara, nCur );
                 }
             }
@@ -165,7 +161,7 @@
     aOptions.Locale = Application::GetSettings().GetLanguageTag().getLocale();
     utl::TextSearch aSearcher( rSearchOptions );
 
-    // ueber die Absaetze iterieren...
+    // iterate over the paragraphs
     for ( sal_uLong nNode = nStartNode;
             bForward ?  ( nNode <= nEndNode) : ( nNode >= nEndNode );
             bForward ? nNode++ : nNode-- )
@@ -199,7 +195,6 @@
             rSel.GetStart().GetIndex() = nStartPos;
             rSel.GetEnd().GetPara() = nNode;
             rSel.GetEnd().GetIndex() = nEndPos;
-            // Ueber den Absatz selektieren?
             // Select over the paragraph?
             // FIXME  This should be max long...
             if( nEndPos == sal::static_int_cast<sal_uInt16>(-1) ) // sal_uInt16 for 0 and -1 !
@@ -219,7 +214,7 @@
             break;
         }
 
-        if ( !bForward && !nNode )  // Bei rueckwaertsuche, wenn nEndNode = 0:
+        if ( !bForward && !nNode )  // if searching backwards, if nEndNode == 0:
             break;
     }
 
@@ -263,8 +258,8 @@
     if ( ((ExtTextEngine*)GetTextEngine())->Search( aSel, rSearchOptions, bForward ) )
     {
         bFound = sal_True;
-        // Erstmal den Anfang des Wortes als Selektion einstellen,
-        // damit das ganze Wort in den sichtbaren Bereich kommt.
+        // First add the beginning of the word to the selection,
+        // so that the whole word is in the visible region.
         SetSelection( aSel.GetStart() );
         ShowCursor( sal_True, sal_False );
     }
@@ -289,7 +284,7 @@
         {
             InsertText( rSearchOptions.replaceString );
             nFound = 1;
-            Search( rSearchOptions, bForward ); // gleich zum naechsten
+            Search( rSearchOptions, bForward ); // right away to the next
         }
         else
         {
@@ -299,7 +294,7 @@
     }
     else
     {
-        // Der Writer ersetzt alle, vom Anfang bis Ende...
+        // the writer replaces all, from beginning to end
 
         ExtTextEngine* pTextEngine = (ExtTextEngine*)GetTextEngine();
 
@@ -351,20 +346,20 @@
     sal_uLong nEndPara = aSel.GetEnd().GetPara();
     if ( aSel.HasRange() && !aSel.GetEnd().GetIndex() )
     {
-        nEndPara--; // den dann nicht einruecken...
+        nEndPara--; // do not indent
     }
 
     for ( sal_uLong nPara = nStartPara; nPara <= nEndPara; nPara++ )
     {
         if ( bRight )
         {
-            // Tabs hinzufuegen
+            // add tabs
             GetTextEngine()->ImpInsertText( TextPaM( nPara, 0 ), '\t' );
             bDone = sal_True;
         }
         else
         {
-            // Tabs/Blanks entfernen
+            // remove Tabs/Blanks
             String aText = GetTextEngine()->GetText( nPara );
             if ( aText.Len() && (
                     ( aText.GetChar( 0 ) == '\t' ) ||

-- 
To view, visit https://gerrit.libreoffice.org/3787
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6cb657853cf3e8d4e35c0dc09be6a4a15ba3a336
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Christian M. Heller <christian.heller63 at gmail.com>



More information about the LibreOffice mailing list