Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


Hi,

Here's a patch who removes bogus comments. It's released under the
LGPLv3+ / MPL.

Antoine

P.-S. : I'm very poor in English (I speak French), but I try to do the
best I can to write in this language.
From 14a9700e569847c46f8d982ff629c3de9e661a56 Mon Sep 17 00:00:00 2001
From: Antoine Proulx <proulx.antoine@gmail.com>
Date: Sat, 5 Feb 2011 17:11:34 -0500
Subject: [PATCH] Remove bogus comments

---
 sw/source/core/view/pagepreviewlayout.cxx |   20 ++++++++++----------
 sw/source/core/view/viewsh.cxx            |   11 ++++-------
 sw/source/core/view/vprint.cxx            |    3 +--
 3 files changed, 15 insertions(+), 19 deletions(-)

diff --git a/sw/source/core/view/pagepreviewlayout.cxx b/sw/source/core/view/pagepreviewlayout.cxx
index ec64a9b..ad64cba 100644
--- a/sw/source/core/view/pagepreviewlayout.cxx
+++ b/sw/source/core/view/pagepreviewlayout.cxx
@@ -254,7 +254,7 @@ void SwPagePreviewLayout::_ApplyNewZoomAtViewShell( sal_uInt8 _aNewZoom )
     if ( aNewViewOptions.GetZoom() != _aNewZoom )
     {
         aNewViewOptions.SetZoom( _aNewZoom );
-        // OD 24.09.2003 #i19975# - consider zoom type.
+        //#i19975# - consider zoom type.
         enum SvxZoomType eZoomType = SVX_ZOOM_PERCENT;
         aNewViewOptions.SetZoomType( eZoomType );
         mrParentViewShell.ApplyViewOptions( aNewViewOptions );
@@ -750,7 +750,7 @@ Point SwPagePreviewLayout::GetPreviewStartPosForNewScale(
 
 /** determines, if page with given page number is visible in preview
 
-    OD 12.12.2002 #103492#
+    #103492#
 
     @author OD, _nPageNum is absolut!
 */
@@ -762,7 +762,7 @@ bool SwPagePreviewLayout::IsPageVisible( const sal_uInt16 _nPageNum ) const
 
 /** calculate data to bring new selected page into view.
 
-    OD 12.12.2002 #103492#
+    #103492#
 
     @author OD, IN/OUT parameters are absolute page numbers!!!
 */
@@ -777,7 +777,7 @@ bool SwPagePreviewLayout::CalcStartValuesForSelectedPageMove(
     sal_uInt16 nTmpRelSelPageNum = ConvertAbsoluteToRelativePageNum( mnSelectedPageNum );
     sal_uInt16 nNewRelSelectedPageNum = nTmpRelSelPageNum;
 
-    // OD 19.02.2003 #107369# - leaving left-top-corner blank is controlled
+    // #107369# - leaving left-top-corner blank is controlled
     // by <mbBookPreview>.
     if ( mbBookPreview )
     {
@@ -828,13 +828,13 @@ bool SwPagePreviewLayout::CalcStartValuesForSelectedPageMove(
         sal_Int16 nTotalRows = GetRowOfPage( mnPages );
         if ( (_nHoriMove > 0 || _nVertMove > 0) &&
              mbDoesLayoutRowsFitIntoWindow &&
-             mbDoesLayoutColsFitIntoWindow && // OD 20.02.2003 #107369# - add condition
+             mbDoesLayoutColsFitIntoWindow && // #107369# - add condition
              nCurrRow > nTotalRows - mnRows )
         {
             // new proposed start page = left-top-corner of last possible
             // preview page.
             nNewStartPage = (nTotalRows - mnRows) * mnCols + 1;
-            // OD 19.02.2003 #107369# - leaving left-top-corner blank is controlled
+            // #107369# - leaving left-top-corner blank is controlled
             // by <mbBookPreview>.
             if ( mbBookPreview )
             {
@@ -860,7 +860,7 @@ bool SwPagePreviewLayout::CalcStartValuesForSelectedPageMove(
 
 /** checks, if given position is inside a shown document page
 
-    OD 17.12.2002 #103492#
+    #103492#
 
     @author OD
 */
@@ -924,7 +924,7 @@ bool SwPagePreviewLayout::IsPrevwPosInDocPrevwPage( const Point  _aPrevwPos,
 
 /** determine window page scroll amount
 
-    OD 17.12.2002 #103492#
+    #103492#
 
     @author OD
 */
@@ -939,14 +939,14 @@ SwTwips SwPagePreviewLayout::GetWinPagesScrollAmount(
     else
         nScrollAmount = _nWinPagesToScroll * maPaintedPrevwDocRect.GetHeight();
 
-    // OD 19.02.2003 #107369# - check, if preview layout size values are valid.
+    // #107369# - check, if preview layout size values are valid.
     // If not, the checks for an adjustment of the scroll amount aren't useful.
     if ( mbLayoutSizesValid )
     {
         if ( (maPaintedPrevwDocRect.Top() + nScrollAmount) <= 0 )
             nScrollAmount = -maPaintedPrevwDocRect.Top();
 
-        // OD 14.02.2003 #107369# - correct scroll amount
+        // #107369# - correct scroll amount
         if ( nScrollAmount > 0 &&
              maPaintedPrevwDocRect.Bottom() == maPreviewDocRect.Bottom()
            )
diff --git a/sw/source/core/view/viewsh.cxx b/sw/source/core/view/viewsh.cxx
index 72dcebc..59cf310 100644
--- a/sw/source/core/view/viewsh.cxx
+++ b/sw/source/core/view/viewsh.cxx
@@ -331,7 +331,7 @@ void ViewShell::ImplEndAction( const BOOL bIdleEnd )
                         DLPostPaint2(true);
                     }
 
-                    // --> OD 2009-12-03 #i107365#
+                    // #i107365#
                     // Direct paint has been performed. Thus, take care of
                     // transparent child windows.
                     if ( GetWin() )
@@ -360,7 +360,6 @@ void ViewShell::ImplEndAction( const BOOL bIdleEnd )
                             }
                         }
                     }
-                    // <--
                 }
 
                 delete pVout;
@@ -666,9 +665,7 @@ void lcl_InvalidateAllCntnt( ViewShell& rSh, BYTE nInv )
     objects (Writer fly frame and drawing objects), which are anchored
     to paragraph or to character.
 
-    OD 2004-03-16 #i11860#
-
-    @author OD
+    #i11860#
 */
 void lcl_InvalidateAllObjPos( ViewShell &_rSh )
 {
@@ -1665,7 +1662,7 @@ void ViewShell::Paint(const Rectangle &rRect)
         return;
     }
 
-    //MA 30. Jul. 95: fix(16787): mit !nStartAction versuche ich mal mich gegen
+    //fix(16787): mit !nStartAction versuche ich mal mich gegen
     //fehlerhaften Code an anderen Stellen zu wehren. Hoffentlich fuehrt das
     //nicht zu Problemen!?
     if ( bPaintWorks && !nStartAction )
@@ -2244,7 +2241,7 @@ void ViewShell::ApplyAccessiblityOptions(SvtAccessibilityOptions& 
rAccessibility
         pAccOptions->SetStopAnimatedGraphics(! rAccessibilityOptions.GetIsAllowAnimatedGraphics());
         pAccOptions->SetStopAnimatedText(! rAccessibilityOptions.GetIsAllowAnimatedText());
 
-        // --> FME 2004-06-29 #114856# Formular view
+        // #114856# Formular view
         // Always set this option, not only if document is read-only:
         pOpt->SetSelectionInReadonly(rAccessibilityOptions.IsSelectionInReadonly());
     }
diff --git a/sw/source/core/view/vprint.cxx b/sw/source/core/view/vprint.cxx
index b37a383..295bbf3 100644
--- a/sw/source/core/view/vprint.cxx
+++ b/sw/source/core/view/vprint.cxx
@@ -684,7 +684,7 @@ void ViewShell::PrtOle2( SwDoc *pDoc, const SwViewOption *pOpt, const SwPrintDat
             pDoc->GetRootFrm()->Lower()->InvalidateSize();
         }
 
-        // --> FME 2005-02-10 #119474#
+        // #119474#
         // CalcPagesForPrint() should not be necessary here. The pages in the
         // visible area will be formatted in SwRootFrm::Paint().
         // Removing this gives us a performance gain during saving the
@@ -692,7 +692,6 @@ void ViewShell::PrtOle2( SwDoc *pDoc, const SwViewOption *pOpt, const SwPrintDat
         // formatting of the document.
         // Seiten fuers Drucken formatieren
         // pSh->CalcPagesForPrint( SHRT_MAX );
-        // <--
         pOleOut->Push( PUSH_CLIPREGION );
         pOleOut->IntersectClipRegion( aSwRect.SVRect() );
         pSh->GetLayout()->Paint( aSwRect );
-- 
1.7.1


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.