From 8ea4d025ded75eee0918258121d54273aead66ad Mon Sep 17 00:00:00 2001 From: zecke Date: Mon, 04 Apr 2005 20:23:39 +0000 Subject: Speed up start times of applications: -Do not load MimeType when using Resource::loadIconSet we introduce a global boolean (like TT) What is bad: The boolean is visible outside of libqpe and we've that extern in the header... On the former we can use visibility attributes soon --- (limited to 'library') diff --git a/library/resource.cpp b/library/resource.cpp index 18139b9..da09892 100644 --- a/library/resource.cpp +++ b/library/resource.cpp @@ -25,6 +25,12 @@ #include #include +/* + * enable or disable the search for the icon without .png or .xpm + * suffix. We would use MimeType to lookup possible extensions... + */ +bool qpe_fast_findPixmap = false; // visible in libqpe + // this namespace is just a workaround for a gcc bug // gcc exports inline functions in the generated file // inlinepics_p.h @@ -65,13 +71,15 @@ QPixmap Resource::loadPixmap( const QString &pix ) QPixmap pm; // null pixmap QString key="QPE_"+pix; if ( !QPixmapCache::find(key,pm) ) { - QImage I = loadImage(pix); - if( I.isNull() ) { + QImage I = loadImage(pix); + if( I.isNull() ) { qWarning( "Could not load %s", pix.latin1() ); } else { pm.convertFromImage(I); QPixmapCache::insert(key,pm); } + }else { + qWarning("In Cache for %s pixmap %s", qApp->argv()[0], pix.local8Bit().data() ); } return pm; } @@ -140,17 +148,20 @@ QString Resource::findPixmap( const QString &pix ) if ( QFile( f ).exists() ) return f; - // All formats... - QStringList exts = opie_imageExtensions(); - for ( QStringList::ConstIterator it = exts.begin(); it!=exts.end(); ++it ) { - QString f = picsPath + pix + "." + *it; - if ( QFile(f).exists() ) - return f; - } + if ( !qpe_fast_findPixmap ) { + printf("Doing slow search for %s %s\n", qApp->argv()[0], pix.local8Bit().data() ); + // All formats... + QStringList exts = opie_imageExtensions(); + for ( QStringList::ConstIterator it = exts.begin(); it!=exts.end(); ++it ) { + QString f = picsPath + pix + "." + *it; + if ( QFile(f).exists() ) + return f; + } - // Finally, no (or existing) extension... - if ( QFile( picsPath + pix ).exists() ) - return picsPath + pix; + // Finally, no (or existing) extension... + if ( QFile( picsPath + pix ).exists() ) + return picsPath + pix; + } //qDebug("Cannot find pixmap: %s", pix.latin1()); return QString(); diff --git a/library/resource.h b/library/resource.h index 1f1ba9a..4a3502a 100644 --- a/library/resource.h +++ b/library/resource.h @@ -42,14 +42,23 @@ public: static QStringList allSounds(); }; +extern bool qpe_fast_findPixmap; // Inline for compatibility with SHARP ROMs -inline QIconSet Resource::loadIconSet( const QString &pix ) +inline QIconSet Resource::loadIconSet( const QString &pix ) { + /* + * disable the slow load + */ + bool oldMode = qpe_fast_findPixmap; + qpe_fast_findPixmap = true; + QPixmap dpm = loadPixmap( pix + "_disabled" ); QPixmap pm = loadPixmap( pix ); QIconSet is( pm ); if ( !dpm.isNull() ) is.setPixmap( dpm, pm.width() <= 22 ? QIconSet::Small : QIconSet::Large, QIconSet::Disabled ); + + qpe_fast_findPixmap = oldMode; return is; } -- cgit v0.9.0.2