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
- [Libreoffice] Easy Hack completed - All bogus comments removed · Júlio Hoffimann
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.