[PATCH] Remove unused debug facilities from basebmp

Jan Holesovsky kendy at suse.cz
Wed Apr 18 08:58:56 PDT 2012


Hi Gabor,

On 2012-04-14 at 11:58 +0200, Lubos Lunak wrote:

> > Remove debug dumping facility from basebmp.
> 
> From the patch:
> 
> "Used in vcl/headless/svpgdi.cxx when OSL_DEBUG_LEVEL > 2"
> 
>  And it indeed is. Moreover it looks like quite useful debugging 
> functionality, so I don't think it should be removed just because 
> unusedcode.easy lists it. It should always be checked whether what's listed 
> should really be removed, the file is just automatically generated and it is 
> fallible.

Actually, in cases like this, it is best to enclose the "offending" code
with #if OSL_DEBUG_LEVEL > 2 too, so that it is compiled only when it is
supposed to be called - can you update your patch that way, please?

Thank you,
Kendy



More information about the LibreOffice mailing list