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


Hi friends,

This patch removes all bogus comments laying around. If you are searching
for one of the following, you could stop. ;-)

/* -----------------------------04.12.00 16:26--------------------------------

and

#include <svx/svxdlg.hxx> //CHINA001

and

// --> OD 2008-01-08 #newlistlevelattrs#


I did it using regex, so one or another replacement discards a piece of
comment when it is in a block, for example:

// --> OD 2008-01-08 #i1234# first line comment
// continuing in the second line
// ...

The regex will remove "first line comment" in that case, but will keep the
Bug ID as Thorsten requested me in another email.

Could i mark this Easy Hack as completed in the Wiki?

Regards,
Júlio.
From 58c58d23c3f5f089b8d640c6791bf713ecb24e2e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlio=20Hoffimann?= <julio.hoffimann@gmail.com>
Date: Thu, 14 Apr 2011 08:09:11 -0300
Subject: [PATCH] all bogus comments removed by regular expressions

---
 framework/source/layoutmanager/helpers.cxx |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/framework/source/layoutmanager/helpers.cxx b/framework/source/layoutmanager/helpers.cxx
index 652cf15..aeab496 100644
--- a/framework/source/layoutmanager/helpers.cxx
+++ b/framework/source/layoutmanager/helpers.cxx
@@ -339,7 +339,7 @@ sal_Bool implts_isFrameOrWindowTop( const uno::Reference< frame::XFrame >& xFram
     uno::Reference< awt::XTopWindow > xWindowCheck(xFrame->getContainerWindow(), uno::UNO_QUERY); 
// dont use _THROW here ... its a check only
     if (xWindowCheck.is())
     {
-        // --> PB 2007-06-18 #i76867# top and system window is required.
+    // #i76867#
         SolarMutexGuard aGuard;
         uno::Reference< awt::XWindow > xWindow( xWindowCheck, uno::UNO_QUERY );
         Window* pWindow = VCLUnoHelper::GetWindow( xWindow );
-- 
1.7.0.4

From 5c755c4bc25cd8c0d703124de4bf85a2003dca89 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlio=20Hoffimann?= <julio.hoffimann@gmail.com>
Date: Thu, 14 Apr 2011 08:12:27 -0300
Subject: [PATCH] all bogus comments removed by regular expressions

---
 comphelper/source/misc/accessiblewrapper.cxx |    2 +-
 svtools/source/control/ruler.cxx             |    2 +-
 toolkit/source/helper/unowrapper.cxx         |    2 +-
 vcl/inc/vcl/ptrstyle.hxx                     |    4 ++--
 vcl/inc/vcl/vclevent.hxx                     |    2 +-
 vcl/os2/source/window/salframe.cxx           |    4 ++--
 vcl/unx/gtk/a11y/atklistener.cxx             |    2 +-
 vcl/unx/gtk/a11y/atktextattributes.cxx       |    2 +-
 vcl/unx/gtk/app/gtkdata.cxx                  |    4 ++--
 vcl/win/source/gdi/winlayout.cxx             |    4 ++--
 vcl/win/source/window/salframe.cxx           |    4 ++--
 11 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/comphelper/source/misc/accessiblewrapper.cxx 
b/comphelper/source/misc/accessiblewrapper.cxx
index 60103cc..8b072d8 100644
--- a/comphelper/source/misc/accessiblewrapper.cxx
+++ b/comphelper/source/misc/accessiblewrapper.cxx
@@ -241,7 +241,7 @@ namespace comphelper
             case AccessibleEventId::TABLE_ROW_DESCRIPTION_CHANGED:
             case AccessibleEventId::TABLE_ROW_HEADER_CHANGED:
             case AccessibleEventId::TABLE_SUMMARY_CHANGED:
-            // --> PB 2006-03-21 #130798# EventId TEXT_SELECTION_CHANGED was missed
+    // #130798#
             // these Ids are also missed: SUB_WINDOW_OF_RELATION_CHANGED & TEXT_ATTRIBUTE_CHANGED
             case AccessibleEventId::TEXT_SELECTION_CHANGED:
             // <--
diff --git a/svtools/source/control/ruler.cxx b/svtools/source/control/ruler.cxx
index a93207d..2202d3e 100644
--- a/svtools/source/control/ruler.cxx
+++ b/svtools/source/control/ruler.cxx
@@ -1679,7 +1679,7 @@ sal_Bool Ruler::ImplHitTest( const Point& rPos, ImplRulerHitTest* pHitTest,
     }
     nHitBottom = mnVirHeight+(RULER_OFF*2);
 
-    // --> FME 2004-08-05 #i32608#
+    // #i32608#
     pHitTest->nAryPos = 0;
     pHitTest->mnDragSize = 0;
     pHitTest->bSize = sal_False;
diff --git a/toolkit/source/helper/unowrapper.cxx b/toolkit/source/helper/unowrapper.cxx
index 54269ce..50b90b3 100644
--- a/toolkit/source/helper/unowrapper.cxx
+++ b/toolkit/source/helper/unowrapper.cxx
@@ -64,7 +64,7 @@ using namespace ::com::sun::star;
         case WINDOW_OKBUTTON:
         case WINDOW_CANCELBUTTON:   return new VCLXButton;
         case WINDOW_CHECKBOX:       return new VCLXCheckBox;
-        // --> OD 2009-06-29 #i95042#
+    // #i95042#
         // A Window of type <MetricBox> is inherited from type <ComboBox>.
         // Thus, it does make more sense to return a <VCLXComboBox> instance
         // instead of only a <VCLXWindow> instance, especially regarding its
diff --git a/vcl/inc/vcl/ptrstyle.hxx b/vcl/inc/vcl/ptrstyle.hxx
index 680c6b7..a1cf690 100644
--- a/vcl/inc/vcl/ptrstyle.hxx
+++ b/vcl/inc/vcl/ptrstyle.hxx
@@ -126,7 +126,7 @@ typedef sal_uInt16 PointerStyle;
 #define POINTER_TEXT_VERTICAL                  ((PointerStyle)86)
 #define POINTER_PIVOT_DELETE                   ((PointerStyle)87)
 
-// --> FME 2004-07-30 #i32329# Enhanced table selection
+    // #i32329#
 #define POINTER_TAB_SELECT_S            ((PointerStyle)88)
 #define POINTER_TAB_SELECT_E            ((PointerStyle)89)
 #define POINTER_TAB_SELECT_SE           ((PointerStyle)90)
@@ -134,7 +134,7 @@ typedef sal_uInt16 PointerStyle;
 #define POINTER_TAB_SELECT_SW           ((PointerStyle)92)
 // <--
 
-// --> FME 2004-08-16 #i20119# Paintbrush tool
+    // #i20119#
 #define POINTER_PAINTBRUSH              ((PointerStyle)93)
 // <--
 
diff --git a/vcl/inc/vcl/vclevent.hxx b/vcl/inc/vcl/vclevent.hxx
index 2e55ce0..777d39e 100644
--- a/vcl/inc/vcl/vclevent.hxx
+++ b/vcl/inc/vcl/vclevent.hxx
@@ -154,7 +154,7 @@ namespace com { namespace sun { namespace star {
 #define VCLEVENT_ROADMAP_ITEMSELECTED       1171
 #define VCLEVENT_TOOLBOX_FORMATCHANGED      1172        // request new layout
 #define VCLEVENT_COMBOBOX_SETTEXT           1173
-// --> OD 2009-04-01 #i92103#
+    // #i92103#
 #define VCLEVENT_ITEM_EXPANDED              1174
 #define VCLEVENT_ITEM_COLLAPSED             1175
 // <--
diff --git a/vcl/os2/source/window/salframe.cxx b/vcl/os2/source/window/salframe.cxx
index f4a4226..8ef8abe 100644
--- a/vcl/os2/source/window/salframe.cxx
+++ b/vcl/os2/source/window/salframe.cxx
@@ -1536,7 +1536,7 @@ void Os2SalFrame::SetPointer( PointerStyle ePointerStyle )
     { 0, 0, SAL_RESID_POINTER_TEXT_VERTICAL },      // POINTER_TEXT_VERTICAL
     { 0, 0, SAL_RESID_POINTER_PIVOT_DELETE },       // POINTER_PIVOT_DELETE
 
-    // --> FME 2004-07-30 #i32329# Enhanced table selection
+    // #i32329#
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_S },       // POINTER_TAB_SELECT_S
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_E },       // POINTER_TAB_SELECT_E
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_SE },      // POINTER_TAB_SELECT_SE
@@ -1544,7 +1544,7 @@ void Os2SalFrame::SetPointer( PointerStyle ePointerStyle )
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_SW },      // POINTER_TAB_SELECT_SW
     // <--
 
-    // --> FME 2004-08-16 #i20119# Paintbrush tool
+    // #i20119#
     { 0, 0, SAL_RESID_POINTER_PAINTBRUSH }          // POINTER_PAINTBRUSH
     // <--
     };
diff --git a/vcl/unx/gtk/a11y/atklistener.cxx b/vcl/unx/gtk/a11y/atklistener.cxx
index 513c80a..88910a4 100644
--- a/vcl/unx/gtk/a11y/atklistener.cxx
+++ b/vcl/unx/gtk/a11y/atklistener.cxx
@@ -364,7 +364,7 @@ void AtkListener::notifyEvent( const accessibility::AccessibleEventObject& aEven
             break;
         }
 
-        // --> OD 2009-05-26 #i92103#
+    // #i92103#
         case accessibility::AccessibleEventId::LISTBOX_ENTRY_EXPANDED:
         {
             AtkObject *pChild = getObjFromAny( aEvent.NewValue );
diff --git a/vcl/unx/gtk/a11y/atktextattributes.cxx b/vcl/unx/gtk/a11y/atktextattributes.cxx
index bcf8801..c0620cf 100644
--- a/vcl/unx/gtk/a11y/atktextattributes.cxx
+++ b/vcl/unx/gtk/a11y/atktextattributes.cxx
@@ -144,7 +144,7 @@ static const char * ExportedTextAttributes[TEXT_ATTRIBUTE_LAST] =
     "CharStrikeout",        // TEXT_ATTRIBUTE_STRIKETHROUGH
     "CharUnderline",        // TEXT_ATTRIBUTE_UNDERLINE
     "CharWeight",           // TEXT_ATTRIBUTE_WEIGHT
-    // --> OD 2010-03-05 #i92233#
+    // #i92233#
     "MMToPixelRatio",       // TEXT_ATTRIBUTE_MM_TO_PIXEL_RATIO
     // <--
     "ParaAdjust",           // TEXT_ATTRIBUTE_JUSTIFICATION
diff --git a/vcl/unx/gtk/app/gtkdata.cxx b/vcl/unx/gtk/app/gtkdata.cxx
index 64e8641..7df47f6 100644
--- a/vcl/unx/gtk/app/gtkdata.cxx
+++ b/vcl/unx/gtk/app/gtkdata.cxx
@@ -465,7 +465,7 @@ GdkCursor *GtkSalDisplay::getCursor( PointerStyle ePointerStyle )
             MAKE_CURSOR( POINTER_AIRBRUSH, airbrush_ );
             MAKE_CURSOR( POINTER_TEXT_VERTICAL, vertcurs_ );
 
-            // --> FME 2004-07-30 #i32329# Enhanced table selection
+    // #i32329#
             MAKE_CURSOR( POINTER_TAB_SELECT_S, tblsels_ );
             MAKE_CURSOR( POINTER_TAB_SELECT_E, tblsele_ );
             MAKE_CURSOR( POINTER_TAB_SELECT_SE, tblselse_ );
@@ -473,7 +473,7 @@ GdkCursor *GtkSalDisplay::getCursor( PointerStyle ePointerStyle )
             MAKE_CURSOR( POINTER_TAB_SELECT_SW, tblselsw_ );
             // <--
 
-            // --> FME 2004-08-16 #i20119# Paintbrush tool
+    // #i20119#
             MAKE_CURSOR( POINTER_PAINTBRUSH, paintbrush_ );
             // <--
 
diff --git a/vcl/win/source/gdi/winlayout.cxx b/vcl/win/source/gdi/winlayout.cxx
index b1fc932..ba8db9c 100644
--- a/vcl/win/source/gdi/winlayout.cxx
+++ b/vcl/win/source/gdi/winlayout.cxx
@@ -1541,7 +1541,7 @@ bool UniscribeLayout::LayoutText( ImplLayoutArgs& rArgs )
                         {
                             if( mpLogClusters[ c ] == i )
                             {
-                                // --> HDU/FME 2005-10-25 #i55716# skip WORDJOINER
+    // #i55716#
                                 if( rArgs.mpStr[ c ] == 0x2060 )
                                     mpOutGlyphs[ i + rVisualItem.mnMinGlyphPos ] = 1;
                                 else
@@ -1557,7 +1557,7 @@ bool UniscribeLayout::LayoutText( ImplLayoutArgs& rArgs )
                         {
                             if( mpLogClusters[ c ] == i )
                             {
-                                // --> HDU/FME 2005-10-25 #i55716# skip WORDJOINER
+    // #i55716#
                                 if( rArgs.mpStr[ c ] == 0x2060 )
                                     mpOutGlyphs[ i + rVisualItem.mnMinGlyphPos ] = 1;
                                 else
diff --git a/vcl/win/source/window/salframe.cxx b/vcl/win/source/window/salframe.cxx
index 4ee94b3..609d48e 100644
--- a/vcl/win/source/window/salframe.cxx
+++ b/vcl/win/source/window/salframe.cxx
@@ -2289,7 +2289,7 @@ void WinSalFrame::SetPointer( PointerStyle ePointerStyle )
     { 0, 0, SAL_RESID_POINTER_TEXT_VERTICAL },      // POINTER_TEXT_VERTICAL
     { 0, 0, SAL_RESID_POINTER_PIVOT_DELETE },       // POINTER_PIVOT_DELETE
 
-    // --> FME 2004-07-30 #i32329# Enhanced table selection
+    // #i32329#
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_S },       // POINTER_TAB_SELECT_S
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_E },       // POINTER_TAB_SELECT_E
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_SE },      // POINTER_TAB_SELECT_SE
@@ -2297,7 +2297,7 @@ void WinSalFrame::SetPointer( PointerStyle ePointerStyle )
     { 0, 0, SAL_RESID_POINTER_TAB_SELECT_SW },      // POINTER_TAB_SELECT_SW
     // <--
 
-    // --> FME 2004-08-16 #i20119# Paintbrush tool
+    // #i20119#
     { 0, 0, SAL_RESID_POINTER_PAINTBRUSH }          // POINTER_PAINTBRUSH
     // <--
 
-- 
1.7.0.4


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.