Re: [PUSHED][PATCH] unusedcode.easy: Removed unused code

2012-02-15 Thread Caolán McNamara 
On Fri, 2012-02-10 at 17:39 +0100, Alexander Bergmann wrote:
> ---
>  editeng/source/editeng/editdoc.cxx |8 
>  editeng/source/editeng/editdoc.hxx |1 -

pushed this now, thanks for this.

C.

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

[PUSHED][PATCH] unusedcode.easy: Removed unused code

2012-02-13 Thread Caolán McNamara 
On Fri, 2012-02-10 at 15:29 +0100, Alexander Bergmann wrote:
>  15 files changed, 0 insertions(+), 217 deletions(-)

Pushed this now, thanks for this. Though SetEnableATToolSupport is
actually used under windows, so I tweaked it to be windows-only code
rather than remove it.

C.

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

[PUSHED]PATCH] unusedcode.easy: Removed unused code

2012-02-10 Thread Caolán McNamara 
On Thu, 2012-02-09 at 21:18 +0100, Alexander Bergmann wrote:
> ---
>  .../source/drivers/postgresql/pq_tools.cxx |   14 ---
>  .../source/drivers/postgresql/pq_tools.hxx |2 -
>  pyuno/inc/pyuno/pyuno.hxx  |   11 --
>  pyuno/source/module/pyuno_impl.hxx |2 -
>  pyuno/source/module/pyuno_runtime.cxx  |   17 ---
>  pyuno/source/module/pyuno_type.cxx |  104 
> 
>  sax/inc/sax/tools/converter.hxx|   12 ---
>  sax/source/tools/converter.cxx |   76 --
>  sdext/source/pdfimport/inc/pdfparse.hxx|1 -
>  sdext/source/pdfimport/pdfparse/pdfparse.cxx   |   49 -
>  unusedcode.easy|   10 --

Pushed this now, thanks for this. I guess its possible that the pyuno
ones *should* be called from something, but I can't tell otherwise, so
fine to remove them I guess.

C.

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

[PUSHED][PATCH] unusedcode.easy: Removed unused code (linguistic)

2012-02-09 Thread Caolán McNamara 
On Wed, 2012-02-08 at 12:29 +0100, Alexander Bergmann wrote:
> ---
>  linguistic/inc/linguistic/misc.hxx |6 
>  linguistic/source/misc.cxx |   51 
> 
>  unusedcode.easy|5 ---
>  3 files changed, 0 insertions(+), 62 deletions(-)

Michael pushed this yesterday, marking thread as PUSHED.

C.

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

[Libreoffice] [PUSHED][PATCH] unusedcode.easy: Removed unused code (basegfx)

2012-02-01 Thread Caolán McNamara 
On Wed, 2012-02-01 at 10:16 +, Noel Power wrote:
> perhaps there is a callcatcher special case at work..

Anyway, fixed this up and pushed.

The special case is that callcatcher is a simple-minded compiler
wrapper, and only sees stuff that gcc emits code for. In these cases
gcc, even with -O0, doesn't emit code for inline methods that aren't
called, so while those inlines reference the "unused methods" no code
for the inlines are emitted either, i.e. the inlines that failed to
compile are *themselves* also unused.

I consider it a fun feature.

C.

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