- remove old Icons from the taskbar (only when you start a new app )

2 files changed, 38 insertions, 6 deletions

diff --git a/core/launcher/launcher.cpp b/core/launcher/launcher.cpp
index cf1a3c8..17a30e6 100644
--- a/core/launcher/launcher.cpp
+++ b/core/launcher/launcher.cpp
@@ -547,97 +547,97 @@ void Launcher::loadDocs() // ok here comes a hack belonging to Global::
     tmp = new DocLnkSet( home , QString::null);
     docsFolder->appendFrom( *tmp );
     delete tmp;
     // find out wich filesystems are new in this round
     // We will do this by having a timestamp inside each mountpoint
     // if the current timestamp doesn't match this is a new file system and 
     // come up with our MediumMountGui :) let the hacking begin
     int stamp = uidgen.generate();
     
     QString newStamp = QString::number( stamp ); // generates newtime Stamp
     StorageInfo storage;
     const QList<FileSystem> &fileSystems = storage.fileSystems();
     QListIterator<FileSystem> it ( fileSystems );
     for ( ; it.current(); ++it ) {
       if ( (*it)->isRemovable() ) { // let's find out  if we should search on it
         qWarning("%s is removeable", (*it)->path().latin1() );
         OConfig cfg( (*it)->path() + "/.opiestorage.cf");
         cfg.setGroup("main");
         QString stamp = cfg.readEntry("timestamp", QDateTime::currentDateTime().toString() );
         if( stamp == m_timeStamp ){ // ok we know this card
           cfg.writeEntry("timestamp", newStamp ); //just write a new timestamp
           // we need to scan the list now. Hopefully the cache will be there
           // read the mimetypes from the config and search for documents
           QStringList mimetypes = configToMime( &cfg);
           tmp = new DocLnkSet( (*it)->path(), mimetypes.join(";")  );
           docsFolder->appendFrom( *tmp );
           delete tmp;
 
         }else{ // come up with the gui cause this a new card
           MediumMountGui medium((*it)->path() );
           if( medium.check() ){ // we did not ask before or ask again is off
             if( medium.exec()  ){ // he clicked yes so search it
               // speicher
               cfg.read(); // cause of a race we need to reread
               cfg.writeEntry("timestamp", newStamp );
             }// no else
           }else{ // we checked
             // do something different see what we need to do
             // let's see if we should check the device
             cfg.setGroup("main" );
             bool check = cfg.readBoolEntry("autocheck", true );
             if( check ){ // find the documents
               tmp = new DocLnkSet( (*it)->path(), configToMime(&cfg ).join(";") );
               docsFolder->appendFrom( *tmp );
               delete tmp;
             }
           }
         } 
-      }
+       }
     }
     m_timeStamp = newStamp;
 }
 
 void Launcher::updateTabs()
 {
     MimeType::updateApplications(); // ### reads all applnks twice
 
     delete rootFolder;
     rootFolder = new AppLnkSet( MimeType::appsFolderName() );
 
     loadDocs();
 
     tabs->initializeCategories(rootFolder, docsFolder, storage->fileSystems());
 }
 
 void Launcher::updateDocs()
 {
     loadDocs();
     tabs->updateDocs(docsFolder,storage->fileSystems());
 }
 
 void Launcher::viewSelected(const QString& s)
 {
     setCaption( s + tr(" - Launcher") );
 }
 
 void Launcher::nextView()
 {
     tabs->nextTab();
 }
 
 
 void Launcher::select( const AppLnk *appLnk )
 {
     if ( appLnk->type() == "Folder" ) {
   // Not supported: flat is simpler for the user
     } else {
   if ( appLnk->exec().isNull() ) {
       QMessageBox::information(this,tr("No application"),
     tr("<p>No application is defined for this document."
     "<p>Type is %1.").arg(appLnk->type()));
       return;
   }
   tabs->setBusy(TRUE);
   emit executing( appLnk );
   appLnk->execute();
     }
diff --git a/core/launcher/mrulist.cpp b/core/launcher/mrulist.cpp
index 4daf7d2..6066dac 100644
--- a/core/launcher/mrulist.cpp
+++ b/core/launcher/mrulist.cpp
@@ -1,176 +1,208 @@
 /**********************************************************************
+** Copyright (C) 2002 Holger 'zecke' Freyther
 ** Copyright (C) 2000 Trolltech AS.  All rights reserved.
 **
 ** This file is part of Qtopia Environment.
 **
 ** This file may be distributed and/or modified under the terms of the
 ** GNU General Public License version 2 as published by the Free Software
 ** Foundation and appearing in the file LICENSE.GPL included in the
 ** packaging of this file.
 **
 ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
 ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
 **
 ** See http://www.trolltech.com/gpl/ for GPL licensing information.
 **
 ** Contact info@trolltech.com if any conditions of this licensing are
 ** not clear to you.
 **
 **********************************************************************/
 
 #include "mrulist.h"
 
 #include <qpe/global.h>
 #include <qpe/applnk.h>
 #include <qpe/resource.h>
 
 #include <qframe.h>
 #include <qpushbutton.h>
 #include <qtoolbutton.h>
 #include <qpopupmenu.h>
 #include <qpainter.h>
 #include <qwindowsystem_qws.h>
 
 
         QList<MRUList>*MRUList::MRUListWidgets = NULL;
         QList<AppLnk>*MRUList::task = NULL;
 
 
 MRUList::MRUList( QWidget *parent )
        : QFrame( parent ), selected(-1), oldsel(-1)
 {
     setBackgroundMode( PaletteButton );
     if (!MRUListWidgets)
         MRUListWidgets = new QList<MRUList>;
     if (!task)
         task = new QList<AppLnk>;
     MRUListWidgets->append( this );
 }
 
 
 MRUList::~MRUList()
 {
     if (MRUListWidgets)
         MRUListWidgets->remove( this );
     if (task)
         task->setAutoDelete( TRUE );
 }
 
 
 QSize MRUList::sizeHint() const
 {
     return QSize( frameWidth(), 16 );
 }
 
 
 void MRUList::addTask( const AppLnk *appLnk )
 {
     if ( !appLnk )
         return;
     unsigned int i = 0;
 
     if ( !task )
         return;
-
+    // ok we wan't to delete old icons from the taskbar
+    // get the window list and see which windows aren't there any more
+    QList<AppLnk> cleanUp;
+    cleanUp.setAutoDelete( TRUE );
+    const QList<QWSWindow> &list = qwsServer->clientWindows();
+    QWSWindow* w;
+    bool running = false; // to see what we should do
+    for ( ; i < task->count(); i++ ) {
+      AppLnk *t = task->at(i);
+      running = false;
+      for (QListIterator<QWSWindow> it(list); (w=it.current()); ++it) {
+        QString app = w->client()->identity();
+        if( app == t->exec( ) ){
+          running = true;
+          break;
+        } 
+      }
+      if(!running ) { // gues what we do now
+        cleanUp.append( t);
+      }
+    }
+    // no do a clean up of these old icons
+    AppLnk *lnk;
+    for( lnk = cleanUp.first(); lnk != 0; lnk = cleanUp.next() ){
+      task->remove( lnk );
+    }    
+    cleanUp.clear(); // should be deleted too
+
+    i = 0;
     for ( ; i < task->count(); i++ ) {
         AppLnk *t = task->at(i);
         if ( t->exec() == appLnk->exec() ) {
             if (i != 0) {
                 task->remove();
                 task->prepend( t );
             }
             for (unsigned i = 0; i < MRUListWidgets->count(); i++ )
                 MRUListWidgets->at(i)->update();
             return;
         }
     }
-
+    // check which tasks are running and delete them from the list
     AppLnk *t = new AppLnk( *appLnk );
     // DocLnks have an overloaded virtual function exec()
     t->setExec( appLnk->exec() );
     task->prepend( t );
 
     if ( task->count() > 6 ) {
         t = task->last();
         task->remove();
         Global::terminate(t);
         delete t;
     }
 
     for (unsigned i = 0; i < MRUListWidgets->count(); i++ )
         MRUListWidgets->at(i)->update();
 }
 
 bool MRUList::quitOldApps()
 {
     QStringList appsstarted;
     QStringList appsrunning;
     for ( int i=task->count()-1; i>=0; --i ) {
         AppLnk *t = task->at(i);
         appsstarted.append(t->exec());
     }
 
     const QList<QWSWindow> &list = qwsServer->clientWindows();
     QWSWindow* w;
     for (QListIterator<QWSWindow> it(list); (w=it.current()); ++it) {
-        QString app = w->client()->identity();
+      QString app = w->client()->identity();
         if ( appsstarted.contains(app) && !appsrunning.contains(app) )
             appsrunning.append(app);
     }
 
     if ( appsrunning.count() > 1 ) {
         QStringList::ConstIterator it = appsrunning.begin();
         ++it; // top stays running!
         for (; it != appsrunning.end(); it++) {
             for ( int i=task->count()-1; i>=0; --i ) {
                 AppLnk *t = task->at(i);
-                if ( t->exec() == *it )
-                    Global::terminate(t);
+                if ( t->exec() == *it ){
+                  task->remove(i );
+                  delete t;
+                  Global::terminate(t);
+                }
             }
         }
         return TRUE;
     } else {
         return FALSE;
     }
 }
 
 
 void MRUList::mousePressEvent(QMouseEvent *e)
 {
     selected = 0;
     int x=0;
     QListIterator<AppLnk> it( *task );
     for ( ; it.current(); ++it,++selected,x+=15 ) {
         if ( x + 15 <= width() ) {
             if ( e->x() >= x && e->x() < x+15 ) {
                 if ( selected < (int)task->count() ) {
                     repaint(FALSE);
                     return;
                 }
             }
         } else {
             break;
         }
     }
     selected = -1;
     repaint( FALSE );
 }
 
 
 void MRUList::mouseReleaseEvent(QMouseEvent *)
 {
     if ( selected >= 0 ) {
         if ( parentWidget() )
             if ( parentWidget()->isA( "QPopupMenu" ) )
                 parentWidget()->hide();
         Global::execute( task->at(selected)->exec() );
         selected = -1;
         oldsel = -1;
         update();
     }
 }
 
 
 void MRUList::paintEvent( QPaintEvent * )
 {
     QPainter p( this );