KICONLOADER_CHECKS still necessary?

Michael Pyne mpyne at kde.org
Thu Sep 17 23:03:29 BST 2009


On Thursday 17 September 2009 02:03:04 Boudewijn Rempt wrote:
> On Wednesday 16 September 2009, David Faure wrote:
> > Guess why this code exists? Because we were never able to find a
> >  reproduceable way to trigger #68528. This is a sentinel so that we get
> >  information in the case where the corruption happens.
> >
> > Hopefully KICONLOADER_CHECKS is only enabled in debug mode anyway?
> 
> There is no conditional for that. And it runs in the kde I got from the
> opensuse repositories.

There's no conditional at all, it is enabled unconditionally on 
kiconloader.cpp:368.  Wouldn't be hard to enable only in debug mode if we 
still need it though.

Do we still need it?

Regards,
 - Michael Pyne
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090917/7040e90c/attachment.sig>


More information about the kde-core-devel mailing list