speed up cat selection

2 files changed, 11 insertions, 4 deletions

diff --git a/noncore/graphics/opie-eye/impl/doc/doc_lister.cpp b/noncore/graphics/opie-eye/impl/doc/doc_lister.cpp
index d60149f..e43baed 100644
--- a/noncore/graphics/opie-eye/impl/doc/doc_lister.cpp
+++ b/noncore/graphics/opie-eye/impl/doc/doc_lister.cpp
@@ -25,55 +25,58 @@ Doc_DirLister::Doc_DirLister()
     : PDirLister( "doc_dir_lister" )
 {
     /* connect the signals */
     m_catFilter = 0;
     SlaveMaster* master = SlaveMaster::self();
     connect( master, SIGNAL(sig_start()), this, SIGNAL(sig_start()) );
     connect( master, SIGNAL(sig_end()),   this, SIGNAL(sig_end()) );
     connect( master, SIGNAL(sig_thumbInfo(const QString&, const QString&)),
              this, SLOT(slotThumbInfo(const QString&, const QString&)) );
     connect( master, SIGNAL(sig_fullInfo(const QString&, const QString&)),
              this, SLOT(slotFullInfo(const QString&, const QString&)) );
     connect( master, SIGNAL(sig_thumbNail(const QString&, const QPixmap&)),
              this, SLOT(slotThumbNail(const QString&, const QPixmap&)) );
 
     m_namemap.clear();
     m_filemap.clear();
+    m_docreads = false;
 }
 
 QString Doc_DirLister::defaultPath()const {
     return QString::null;
 }
 
 bool Doc_DirLister::matchCat(const AppLnk* app)
 {
     if (!app) return false;
     if (m_catFilter==0 || app->categories().contains(m_catFilter) || m_catFilter == -1 && app->categories().count() == 0 ) {
         return true;
     }
     return false;
 }
 
 QString Doc_DirLister::setStartPath(const QString&) {
     static const QString Mtype_str("image/jpeg;image/gif;image/bmp;image/png");
     if (m_namemap.isEmpty()) {
-        DocLnkSet ds;
-        Global::findDocuments(&ds,Mtype_str);
-        QListIterator<DocLnk> dit(ds.children());
+        if (!m_docreads) {
+            Global::findDocuments(&m_ds,Mtype_str);
+            m_docreads = true;
+        }
+        QListIterator<DocLnk> dit(m_ds.children());
         for( ; dit.current(); ++dit) {
-            if (! (*dit)->isValid()) continue;
+//            if (! (*dit)->isValid()) continue;
             if (!matchCat((*dit))) continue;
             m_namemap[(*dit)->name()]=(*dit)->file();
             m_filemap[(*dit)->file()]=(*dit)->name();
         }
     }
     return QString::null;
 }
 
 QString Doc_DirLister::currentPath()const {
       return QString::null;
 }
 
 
 QStringList Doc_DirLister::folders()const {
     return QStringList();
 }
diff --git a/noncore/graphics/opie-eye/impl/doc/doc_lister.h b/noncore/graphics/opie-eye/impl/doc/doc_lister.h
index e148ac0..403241c 100644
--- a/noncore/graphics/opie-eye/impl/doc/doc_lister.h
+++ b/noncore/graphics/opie-eye/impl/doc/doc_lister.h
@@ -1,49 +1,53 @@
 /*
  * GPLv2 zecke@handhelds.org
  */
 
 #ifndef DOC_LISTER_INTERFACE_LISTER_H
 #define DOC_LISTER_INTERFACE_LISTER_H
 
 #include <iface/dirlister.h>
 
+#include <qpe/applnk.h>
+
 #include <qstring.h>
 #include <qmap.h>
 
 class Config;
 class AppLnk;
 
 class Doc_DirLister : public PDirLister {
     Q_OBJECT
 public:
     Doc_DirLister();
     virtual ~Doc_DirLister(){}
 
     QString defaultPath()const;
     QString setStartPath( const QString& );
     QString currentPath()const;
     QStringList folders()const;
     QStringList files()const;
 
     void deleteImage( const QString& );
     void thumbNail( const QString&, int, int );
     QImage image( const QString&, Factor, int );
     void imageInfo( const QString& );
     void fullImageInfo( const QString& );
     virtual QString nameToFname(const QString&name)const;
     QString dirUp( const QString& )const;
     QWidget* widget(QWidget*parent);
 
 private:
     QMap<QString,QString> m_namemap,m_filemap;
     int m_catFilter;
     bool matchCat(const AppLnk* app);
+    bool m_docreads;
+    DocLnkSet m_ds;
 
 protected slots:
     virtual void slotFullInfo(const QString&, const QString&);
     virtual void slotThumbInfo(const QString&, const QString&);
     virtual void slotThumbNail(const QString&, const QPixmap&);
     virtual void showCategory(int);
 };
 
 #endif