From kde-commits  Fri May 21 09:09:42 2010
From: David Faure <faure () kde ! org>
Date: Fri, 21 May 2010 09:09:42 +0000
To: kde-commits
Subject: Re: KDE/kdelibs/kdeui/icons
Message-Id: <201005211109.42952.faure () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=127443303018725

On Friday 21 May 2010, Michael Pyne wrote:
> SVN commit 1129007 by mpyne:
> 
> Cache when KIconLoader::loadIcon misses finding an icon.
> 
> My initial port of KIconLoader to KSharedDataCache didn't include the old
> behavior of caching icon search misses. Searching for an icon and not
> finding it is slow! So, cache the results.
> 
> I couldn't really understand what loadIcon was supposed to be doing so I
> spent some time refactoring it instead. The behavior should be compatible
> with sane usage patterns, but my first attempt didn't size "User" icons
> appropriately so look at this patch with trepidation if your icons don't
> work.

Talking about trepidation... why do I bother writing unittests if people who 
refactor code don't even run them?

/d/kde/build/t/kdelibs/kdeui/tests>./kiconloader_unittest
********* Start testing of KIconLoader_UnitTest *********
Config: Using QTest library 4.6.3, Qt 4.6.3
PASS   : KIconLoader_UnitTest::initTestCase()
FAIL!  : KIconLoader_UnitTest::testLoadIconCanReturnNull() 'pix.isNull()' 
returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(66)]
PASS   : KIconLoader_UnitTest::testAppPicsDir()
PASS   : KIconLoader_UnitTest::testAppPicsDir_KIcon()
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(existing icon) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(octet-stream icon) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
QWARN  : KIconLoader_UnitTest::testLoadMimeTypeIcon(non-existing icon) 
QImage::scaled: Image is a null image
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(generic fallback) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(image-x-generic itself) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(x-office-document icon) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
QWARN  : KIconLoader_UnitTest::testLoadMimeTypeIcon(unavailable generic icon) 
QImage::scaled: Image is a null image
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(#184852) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(#178847) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
FAIL!  : KIconLoader_UnitTest::testLoadMimeTypeIcon(mimetype generic icon) 
'path.endsWith(expectedFileName)' returned FALSE. ()
   Loc: [/d/kde/src/t/kdelibs/kdeui/tests/kiconloader_unittest.cpp(176)]
PASS   : KIconLoader_UnitTest::cleanupTestCase()
Totals: 4 passed, 9 failed, 0 skipped
********* Finished testing of KIconLoader_UnitTest *********

This test was passing before your changes.

-- 
David Faure, faure@kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).