-Killl Magic Numbers and introduce an enum

-Split the Command Line on Creation, this way we do not need to
 split the command afterwards

2 files changed, 38 insertions, 27 deletions

diff --git a/core/applets/cardmon/cardmon.cpp b/core/applets/cardmon/cardmon.cpp
index 43ccce2..fb140fe 100644
--- a/core/applets/cardmon/cardmon.cpp
+++ b/core/applets/cardmon/cardmon.cpp
@@ -105,13 +105,14 @@ void CardMonitor::popUp( QString message, QString icon ) {
 void CardMonitor::popupTimeout() {
     popupMenu->hide();
 }
 
 void CardMonitor::mousePressEvent( QMouseEvent * ) {
     QPopupMenu * menu = new QPopupMenu( this );
-    QString cmd;
+    QStringList cmd;
+    bool execute = true;
 
     if ( cardInSd ) {
         menu->insertItem( QIconSet( Resource::loadPixmap( "cardmon/ide" ) ),
                           tr( "Eject SD/MMC card" ), 0 );
     }
 
@@ -132,21 +133,26 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) {
     QPoint p = mapToGlobal( QPoint( 0, 0 ) );
     QSize s = menu->sizeHint();
     int opt = menu->exec( QPoint( p.x() + ( width() / 2 ) - ( s.width() / 2 ),
                                   p.y() - s.height() ), 0 );
 
     if ( opt == 1 ) {
-        m_commandOrig = 1;
-        execCommand("/sbin/cardctl eject 0");
+        m_commandOrig = PCMCIA_Socket1;
+        cmd << "/sbin/cardctl" << "eject" << "0";
     } else if ( opt == 0 ) {
-        m_commandOrig = 2;
-        execCommand( QString("umount %1").arg(cardSdName));
+        m_commandOrig = MMC_Socket;
+        cmd << "umount" << cardSdName;
     } else if ( opt == 2 ) {
-        m_commandOrig = 3;
-        execCommand( "/sbin/cardctl eject 1" );
-    }
+        m_commandOrig = PCMCIA_Socket2;
+        cmd << "/sbin/cardctl" << "eject" <<  "1";
+    }else
+        execute = false;
+
+    if ( execute )
+        execCommand( cmd );
+
     delete menu;
 }
 
 
 void CardMonitor::cardMessage( const QCString & msg, const QByteArray & ) {
     if ( msg == "stabChanged()" ) {
@@ -330,46 +336,45 @@ void CardMonitor::paintEvent( QPaintEvent * ) {
 }
 
 int CardMonitor::position() {
     return 7;
 }
 
-void CardMonitor::execCommand( const QString &command ) {
+void CardMonitor::execCommand( const QStringList &strList ) {
     delete m_process;
     m_process = 0;
 
     if ( m_process == 0 ) {
         m_process = new OProcess();
-        QStringList strList = QStringList::split( " ", command );
 
-           for ( QStringList::Iterator it = strList.begin(); it != strList.end(); ++it ) {
-                *m_process << *it;
-            }
 
-        connect(m_process, SIGNAL( processExited(Opie::Core::OProcess*) ),
-                this, SLOT( slotExited(Opie::Core::OProcess* ) ) );
+        for ( QStringList::ConstIterator it = strList.begin(); it != strList.end(); ++it ) {
+            *m_process << *it;
+        }
+
+        connect(m_process, SIGNAL(processExited(Opie::Core::OProcess*)),
+                this, SLOT( slotExited(Opie::Core::OProcess*)));
 
         if(!m_process->start(OProcess::NotifyOnExit, OProcess::AllOutput ) ) {
             delete m_process;
             m_process = 0;
         }
     }
 }
 
 void CardMonitor::slotExited( OProcess*  ) {
 
    if( m_process->normalExit() ) {
-        int ret = m_process->exitStatus();
-        if( ret != 0 ) {
-            if ( m_commandOrig == 1 ) {
-                popUp( tr( "CF/PCMCIA card eject failed!" ) );
-            } else if ( m_commandOrig == 2 ) {
-                popUp( tr( "SD/MMC card eject failed!" ) );
-            } else if ( m_commandOrig == 3 ) {
-                popUp( tr( "CF/PCMCIA card eject failed!" ) );
-            }
-        }
-    }
+       int ret = m_process->exitStatus();
+       if( ret != 0 ) {
+           if ( m_commandOrig == PCMCIA_Socket1 ||
+                m_commandOrig == PCMCIA_Socket2 ) {
+               popUp( tr( "CF/PCMCIA card eject failed!" ) );
+           } else if ( m_commandOrig == MMC_Socket ) {
+               popUp( tr( "SD/MMC card eject failed!" ) );
+           }
+       }
+   }
 }
 
 EXPORT_OPIE_APPLET_v1( CardMonitor )
 
diff --git a/core/applets/cardmon/cardmon.h b/core/applets/cardmon/cardmon.h
index b02c8b1..400f5ae 100644
--- a/core/applets/cardmon/cardmon.h
+++ b/core/applets/cardmon/cardmon.h
@@ -29,12 +29,18 @@ namespace Opie {
     }
 }
 
 class CardMonitor : public QWidget {
     Q_OBJECT
 public:
+    enum {
+        PCMCIA_Socket1,
+        PCMCIA_Socket2,
+        MMC_Socket
+    };
+
     CardMonitor( QWidget *parent = 0 );
     ~CardMonitor();
     bool getStatusPcmcia( int showPopUp = FALSE );
     bool getStatusSd( int showPopUp = FALSE );
     static int position();
 private slots:
@@ -44,13 +50,13 @@ private slots:
 
 protected:
     void paintEvent( QPaintEvent* );
     void mousePressEvent( QMouseEvent * );
 
 private:
-    void execCommand( const QString &command );
+    void execCommand( const QStringList &command );
     int m_commandOrig;
     QPixmap pm;
     // pcmcia socket 0
     bool cardInPcmcia0;
     QString cardInPcmcia0Name;
     QString cardInPcmcia0Type;