alecs/plasma-framework
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

use a QSet instead of a list since we're doing random access and don't care about order

Browse Source 
svn path=/trunk/KDE/kdelibs/; revision=950827
This commit is contained in:
Aaron J. Seigo 2009-04-07 20:57:19 +00:00
parent 0b5bceb2e9
commit 59a6afb815
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
private/dataengineconsumer_p.h
View File

@ -22,7 +22,9 @@
#ifndef PLASMA_DATAENGINECONSUMER_H #ifndef PLASMA_DATAENGINECONSUMER_H
#define PLASMA_DATAENGINECONSUMER_H #define PLASMA_DATAENGINECONSUMER_H
#include <QtCore/QStringList> #include <QtCore/QSet>
#include <kdebug.h>
#include "plasma/dataenginemanager.h" #include "plasma/dataenginemanager.h"
@ -41,21 +43,20 @@ public:
DataEngine *dataEngine(const QString &name) DataEngine *dataEngine(const QString &name)
{ {
int index = m_loadedEngines.indexOf(name); if (m_loadedEngines.contains(name)) {
if (index != -1) {
return DataEngineManager::self()->engine(name); return DataEngineManager::self()->engine(name);
} }
DataEngine *engine = DataEngineManager::self()->loadEngine(name); DataEngine *engine = DataEngineManager::self()->loadEngine(name);
if (engine->isValid()) { if (engine->isValid()) {
m_loadedEngines.append(name); m_loadedEngines.insert(name);
} }
return engine; return engine;
} }
private: private:
QStringList m_loadedEngines; QSet<QString> m_loadedEngines;
}; };
} // namespace Plasma } // namespace Plasma