Re: [Libreoffice] [PUSHED] Easy Hack completed - All bogus comments removed

2011-04-22 Thread Katarina Machalkova
Hi,

> These are the left arrow comments // <-- if you want to remove them all.
> :-)
> 
> ~1500 lines discarded.
> 
> The big one is in bugzilla:
> https://bugs.freedesktop.org/attachment.cgi?id=45945

Thanks, pushed all of the above (from the previous two messages + the big one 
from bugzilla).
Keep up good work.

B.
-- 
  \  Katarina Machalkova
  \\\__o  LibO developer
__\\\'/_  & hedgehog painter


signature.asc
Description: This is a digitally signed message part.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PUSHED] Easy Hack completed - All bogus comments removed

2011-04-22 Thread Júlio Hoffimann
Hi Cédric,

These are the left arrow comments // <-- if you want to remove them all. :-)

~1500 lines discarded.

The big one is in bugzilla:
https://bugs.freedesktop.org/attachment.cgi?id=45945

Under the LGPLv3+ / MPL license.

Regards,
Júlio.
From 34155f656ff5bab268f65f92e2d2ba1f76868bc5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlio=20Hoffimann?= 
Date: Fri, 22 Apr 2011 07:55:30 -0300
Subject: [PATCH] left arrow comments removed // <--

---
 .../source/extended/accessiblelistbox.cxx  |1 -
 cui/source/dialogs/hltpbase.cxx|2 --
 cui/source/inc/border.hxx  |1 -
 cui/source/tabpages/border.cxx |1 -
 .../star/wizards/document/TimeStampControl.java|2 --
 .../source/dialogs/digitalsignaturesdialog.cxx |1 -
 6 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/accessibility/source/extended/accessiblelistbox.cxx b/accessibility/source/extended/accessiblelistbox.cxx
index 39d986c..c1311f2 100644
--- a/accessibility/source/extended/accessiblelistbox.cxx
+++ b/accessibility/source/extended/accessiblelistbox.cxx
@@ -155,7 +155,6 @@ namespace accessibility
 }
 break;
 }
-// <--
 }
 default:
 VCLXAccessibleComponent::ProcessWindowEvent (rVclWindowEvent);
diff --git a/cui/source/dialogs/hltpbase.cxx b/cui/source/dialogs/hltpbase.cxx
index 095fb1a..846c112 100644
--- a/cui/source/dialogs/hltpbase.cxx
+++ b/cui/source/dialogs/hltpbase.cxx
@@ -453,7 +453,6 @@ IMPL_LINK ( SvxHyperlinkTabPageBase, ClickScriptHdl_Impl, void *, EMPTYARG )
 sal_Bool bIsInputEnabled = GetParent()->IsInputEnabled();
 if ( bIsInputEnabled )
 GetParent()->EnableInput( sal_False );
-// <--
 SfxMacroAssignDlg aDlg( this, mxDocumentFrame, *pItemSet );
 
 // add events
@@ -471,7 +470,6 @@ IMPL_LINK ( SvxHyperlinkTabPageBase, ClickScriptHdl_Impl, void *, EMPTYARG )
 
 if ( bIsInputEnabled )
 GetParent()->EnableInput( sal_True );
-// <--
 // execute dlg
 DisableClose( sal_True );
 short nRet = aDlg.Execute();
diff --git a/cui/source/inc/border.hxx b/cui/source/inc/border.hxx
index eccfed5..b01f71a 100644
--- a/cui/source/inc/border.hxx
+++ b/cui/source/inc/border.hxx
@@ -112,7 +112,6 @@ private:
 CheckBoxaMergeWithNextCB;
 // #i29550#
 CheckBoxaMergeAdjacentBordersCB;
-// <--
 
 ImageList   aShadowImgLstH;
 ImageList   aShadowImgLst;
diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx
index 67046df..a4b54bc 100644
--- a/cui/source/tabpages/border.cxx
+++ b/cui/source/tabpages/border.cxx
@@ -1228,7 +1228,6 @@ void SvxBorderTabPage::PageCreated (SfxAllItemSet aSet)
 aMergeAdjacentBordersCB.Show();
 aPropertiesFL.Show();
 }
-// <--
 }
 if (pFlagItem)
 if ( ( pFlagItem->GetValue() & SVX_HIDESHADOWCTL ) == SVX_HIDESHADOWCTL )
diff --git a/wizards/com/sun/star/wizards/document/TimeStampControl.java b/wizards/com/sun/star/wizards/document/TimeStampControl.java
index 8645f31..558b7da 100644
--- a/wizards/com/sun/star/wizards/document/TimeStampControl.java
+++ b/wizards/com/sun/star/wizards/document/TimeStampControl.java
@@ -150,14 +150,12 @@ public class TimeStampControl extends DatabaseControl
 {
 // --> TESTING
 Point aBeforePt = xShapeGroup.getPosition();
-// <--
 xShapeGroup.setPosition(_aPoint);
 //  oDateControl.xShape.setPosition(_aPoint);
 //  Point atimepoint = new Point(oDateControl.xShape.getPosition().X + oDateControl.xShape.getSize().Width, oDateControl.xShape.getPosition().Y );
 //  oTimeControl.xShape.setPosition(atimepoint);
 // --> TESTING
 Point aAfterPt = xShapeGroup.getPosition();
-// <--
 }
 
 public int getControlType()
diff --git a/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx b/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx
index 8a45f41..0b12a98 100644
--- a/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx
+++ b/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx
@@ -211,7 +211,6 @@ DigitalSignaturesDialog::DigitalSignaturesDialog(
 {
 // --> PB #i48253 the tablistbox needs its own unique id
 maSignaturesLB.Window::SetUniqueId( HID_XMLSEC_TREE_SIGNATURESDLG );
-// <--
 Size aControlSize( maSignaturesLB.GetSizePixel() );
 aControlSize = maSignaturesLB.PixelToLogic( aControlSize, MapMode( MAP_APPFONT ) );
 const long nControlWidth = aControlSize.Width();
-- 
1.7.0.4

From b4078ec4e366db0c2285507c283f8b49097364f2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlio=20Hoffimann?= 
Date: Fri, 22 Apr 2011 08:00:56 -0300
Subject: [PATCH] left arrow comments removed // <--

---
 binfilter/bf_sfx2/source/view/sfx2_printer.cxx |1 -

Re: [Libreoffice] [PUSHED] Easy Hack completed - All bogus comments removed

2011-04-18 Thread Júlio Hoffimann
Hi Cédric,

I'll remove all and the other survivors i found here. :-) Those without the
'i' (#1231#).

Regards,
Júlio.

2011/4/18 Cedric Bosdonnat 

> Hi Julio,
>
> Many thanks for the updated patches. I just pushed them all!
>
> Did you see that you left some empty comments and "// <--" ?
> It would be really awesome if you could prune them ;)
>
> Regards,
> --
> Cedric
>
> On Sat, 2011-04-16 at 23:04 -0300, Júlio Hoffimann wrote:
> > Hi Cédric,
> >
> >
> > I think it's done. :-) The big one is in
> > bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=36234
> >
> >
> > All bogus comments removed keeping the indentation, the bug id and the
> > remainder comment in the same line of the pattern. The next patch will
> > remove the trash in that replacement, e.g. empty comments, closure of
> > SCM comments // <--, etc.
> >
> >
> > Under the LGPLv3+ / MPL license.
> >
> >
> > Regards,
> > Júlio.
> >
> >
> > P.S.: If you want something more, just let me know!
> >
> >
>
> --
> Cédric Bosdonnat
> LibreOffice hacker
> http://documentfoundation.org
> OOo Eclipse Integration developer
> http://cedric.bosdonnat.free.fr
>
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PUSHED] Easy Hack completed - All bogus comments removed

2011-04-18 Thread Cedric Bosdonnat
Hi Julio,

Many thanks for the updated patches. I just pushed them all!

Did you see that you left some empty comments and "// <--" ?
It would be really awesome if you could prune them ;)

Regards,
--
Cedric

On Sat, 2011-04-16 at 23:04 -0300, Júlio Hoffimann wrote:
> Hi Cédric,
> 
> 
> I think it's done. :-) The big one is in
> bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=36234
> 
> 
> All bogus comments removed keeping the indentation, the bug id and the
> remainder comment in the same line of the pattern. The next patch will
> remove the trash in that replacement, e.g. empty comments, closure of
> SCM comments // <--, etc.
> 
> 
> Under the LGPLv3+ / MPL license.
> 
> 
> Regards,
> Júlio.
> 
> 
> P.S.: If you want something more, just let me know!
> 
> 

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice