[PUSHED] Re: [PATCH] Remove unused code (vcl)

2012-04-06 Thread Matúš Kukan
Hello,

On 4 April 2012 17:14, Thomas Collerton tom.col...@gmail.com wrote:
 Hello all,
 this patch removes unused code in the Slider class as listed in
 unusedcode.easy

Looks good = pushed

Thanks,

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


[PUSHED] Re: [PATCH] remove unused code (oox)

2012-04-05 Thread Jan Holesovsky
Hi Petr,

Petr Vorel píše v Po 02. 04. 2012 v 16:18 +0200:

 removed unused classes from oox.

Pushed:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=07ff032c805835355ff0dff75b8ad534dfb770e9

Thank you! :-)

Regards,
Kendy

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


[PUSHED] Re: [PATCH] Remove unused code

2012-02-16 Thread Radek Doulik
Hi Björgvin,

looks good to me. Pushed. Thanks!

Cheers
Radek

On Wed, 2012-02-15 at 01:10 +, Björgvin Ragnarsson wrote:
 Hi,
 
 Attached is a patch for removal of unused code in files
 core/basegfx/source/polygon/b3dpolygontools.cxx and
 core/basegfx/inc/basegfx/polygon/b3dpolygontools.hxx
 
 thanks,
 
 Björgvin
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice


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


[PUSHED] Re: [PATCH] Remove unused code

2012-02-15 Thread Jan Holesovsky
Just adding [PUSHED] :-)

Radek Doulik píše v Út 14. 02. 2012 v 19:03 +0100:
 Hi Santiago,
 
 thanks for you patch, pushed.
 
 Cheers
 Radek
 
 On Mon, 2012-02-13 at 21:55 +0100, Santiago Martinez wrote:
  This patch removes unused code as listed in unusedcode.easy
  
  
  ___
  LibreOffice mailing list
  LibreOffice@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/libreoffice
 
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice


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


[Libreoffice] [PUSHED] Re: [PATCH] Remove unused code

2011-01-11 Thread Caolán McNamara
On Tue, 2011-01-11 at 22:23 +0100, Anders Jonsson wrote:
 Some more commented code, this time in calc. All code has been commented
 out at least since 2005.

Looks good, pushed, thanks for this.

From the patch, it seems that means that ScRedlineOptionsTabPage is now
an empty function that just returns 0 which suggests a follow up patch
to remove it and all references to it, (note IMPL_LINK generally have a
matching DECL_LINK somewhere).

C.

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


[Libreoffice] [PUSHED] Re: [PATCH] Remove unused code and fix typos

2011-01-09 Thread Caolán McNamara
On Sun, 2011-01-09 at 02:06 +0900, Takeshi Abe wrote:
 Hi,
 
 Here is a bunch of cleanup on base.

Looks good to me, pushed, thanks for this.

C.

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


[Libreoffice] [PUSHED] Re: [PATCH] Remove unused code

2010-12-18 Thread Caolán McNamara
On Sat, 2010-12-18 at 14:06 +0100, Anders Jonsson wrote:
 Some more blocks of unused code in libs-gui and libs-core. All have been
 commented out since at least 2006.

All looks good, into the bin with it, thanks for this. Pushed now.

C.

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