Couple minor bits. 1) show the ide icon for sd/mmc, 2) slight change to text on sd/mmc insert/eject popup, 3) do the unmount for sd directly in OZ, as in familiar.

1 files changed, 72 insertions, 46 deletions

diff --git a/core/applets/cardmon/cardmon.cpp b/core/applets/cardmon/cardmon.cpp
index 9293c6f..25bb137 100644
--- a/core/applets/cardmon/cardmon.cpp
+++ b/core/applets/cardmon/cardmon.cpp
@@ -30,4 +30,5 @@
 #include <qtextstream.h>
 #include <qtimer.h>
+#include <qapplication.h>
 
 #include <stdio.h>
@@ -46,13 +47,17 @@ using namespace Opie;
 
 CardMonitor::CardMonitor( QWidget *parent ) : QWidget( parent ),
-                                              pm( Resource::loadPixmap( "cardmon/pcmcia" ) ) {
+pm(Resource::
+   loadPixmap("cardmon/pcmcia"))
+{
 
     QCopChannel* pcmciaChannel = new QCopChannel( "QPE/Card", this );
-    connect( pcmciaChannel, SIGNAL( received( const QCString &, const QByteArray &) ),
-             this, SLOT( cardMessage( const QCString &, const QByteArray &) ) );
+    connect(pcmciaChannel,
+	    SIGNAL(received(const QCString &, const QByteArray &)), this,
+	    SLOT(cardMessage(const QCString &, const QByteArray &)));
 
     QCopChannel* sdChannel = new QCopChannel( "QPE/Card", this );
-    connect( sdChannel, SIGNAL( received( const QCString &, const QByteArray &) ),
-             this, SLOT( cardMessage( const QCString &, const QByteArray &) ) );
+    connect(sdChannel,
+	    SIGNAL(received(const QCString &, const QByteArray &)), this,
+	    SLOT(cardMessage(const QCString &, const QByteArray &)));
 
     cardInPcmcia0 = FALSE;
@@ -67,5 +72,6 @@ CardMonitor::CardMonitor( QWidget *parent ) : QWidget( parent ),
 }
 
-CardMonitor::~CardMonitor() {
+CardMonitor::~CardMonitor()
+{
     if( popupMenu ) {
         delete popupMenu;
@@ -73,5 +79,6 @@ CardMonitor::~CardMonitor() {
 }
 
-void CardMonitor::popUp( QString message, QString icon ) {
+void CardMonitor::popUp(QString message, QString icon)
+{
     if ( ! popupMenu ) {
   popupMenu = new QPopupMenu(this);
@@ -82,11 +89,11 @@ void CardMonitor::popUp( QString message, QString icon ) {
   popupMenu->insertItem( message, 0 );
     } else {
-  popupMenu->insertItem( QIconSet ( Resource::loadPixmap ( icon ) ),  message, 0 );
+	popupMenu->insertItem(QIconSet(Resource::loadPixmap(icon)),
+			      message, 0);
     }
 
     QPoint p = mapToGlobal ( QPoint ( 0, 0 ) );
     QSize s = popupMenu->sizeHint ();
-    popupMenu->popup( QPoint (
-                          p. x ( ) + ( width ( ) / 2 ) - ( s. width ( ) / 2 ),
+    popupMenu->popup(QPoint(p.x() + (width() / 2) - (s.width() / 2),
                           p. y ( ) - s. height ( ) ), 0);
 
@@ -94,9 +101,11 @@ void CardMonitor::popUp( QString message, QString icon ) {
 }
 
-void CardMonitor::popupTimeout() {
+void CardMonitor::popupTimeout()
+{
     popupMenu->hide();
 }
 
-void CardMonitor::mousePressEvent( QMouseEvent * ) {
+void CardMonitor::mousePressEvent(QMouseEvent *)
+{
     QPopupMenu *menu = new QPopupMenu(this);
     QString cmd;
@@ -104,14 +113,19 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) {
 
     if ( cardInSd ) {
-        menu->insertItem( tr("Eject SD/MMC card"), 0 );
+	menu->insertItem(QIconSet(Resource::loadPixmap("cardmon/ide")),
+			tr("Eject SD/MMC card"), 0);
     }
 
     if ( cardInPcmcia0 ) {
-        menu->insertItem( QIconSet ( Resource::loadPixmap ( "cardmon/" + cardInPcmcia0Type ) ),
+	menu->
+	    insertItem(QIconSet
+		       (Resource::loadPixmap("cardmon/" + cardInPcmcia0Type)),
                           tr( "Eject card 0: %1" ).arg(cardInPcmcia0Name ), 1 );
     }
 
     if ( cardInPcmcia1 ) {
-        menu->insertItem( QIconSet ( Resource::loadPixmap ( "cardmon/" + cardInPcmcia1Type ) ),
+	menu->
+	    insertItem(QIconSet
+		       (Resource::loadPixmap("cardmon/" + cardInPcmcia1Type)),
     tr( "Eject card 1: %1" ).arg( cardInPcmcia1Name ), 2 );
     }
@@ -119,6 +133,5 @@ 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 ),
+    int opt = menu->exec(QPoint(p.x() + (width() / 2) - (s.width() / 2),
   p. y ( ) - s. height ( ) ), 0);
 
@@ -126,6 +139,7 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) {
         cmd = "/sbin/cardctl eject 0";
   err = system( ( const char * ) cmd );
-  if ( ( err == 127 ) || ( err < 0 ) ) {
-      qDebug( "Could not execute `/sbin/cardctl eject 0'! err=%d", err );
+	if (err != 0) {
+	    qDebug("Could not execute `/sbin/cardctl eject 0'! err=%d",
+		   err);
       popUp( tr( "CF/PCMCIA card eject failed!" ) );
   }
@@ -133,21 +147,17 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) {
         if ( ODevice::inst()->system() == System_Familiar ) {
             cmd = "umount /dev/mmc/part1";
-            err = system( ( const char *) cmd );
-            if ( ( err != 0 ) ) {
-                popUp( tr("SD/MMC card eject failed!") );
-            }
         } else {
-            cmd = "/etc/sdcontrol compeject";
+	    cmd = "umount /dev/mmcda1";
+	}
             err = system( ( const char *) cmd );
-            if ( ( err != 0 ) ) {
-                qDebug( "Could not execute `/etc/sdcontrol comeject'! err=%d", err );
+	if (err != 0) {
                 popUp( tr("SD/MMC card eject failed!") );
             }
-        }
     } else if ( opt == 2 ) {
         cmd = "/sbin/cardctl eject 1";
         err = system( ( const char *) cmd );
-  if ( ( err == 127 ) || ( err < 0 ) ) {
-      qDebug( "Could not execute `/sbin/cardctl eject 1'! err=%d", err );
+	if (err != 0) {
+	    qDebug("Could not execute `/sbin/cardctl eject 1'! err=%d",
+		   err);
       popUp( tr( "CF/PCMCIA card eject failed!" ) );
   }
@@ -157,15 +167,17 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) {
 
 
-void CardMonitor::cardMessage( const QCString &msg, const QByteArray & ) {
+void CardMonitor::cardMessage(const QCString & msg, const QByteArray &)
+{
     if ( msg == "stabChanged()" ) {
        // qDebug("Pcmcia: stabchanged");
             getStatusPcmcia();
     } else if ( msg == "mtabChanged()" ) {
-        qDebug("CARDMONAPPLET: mtabchanged");
+	// qDebug("CARDMONAPPLET: mtabchanged");
             getStatusSd();
         }
 }
 
-bool CardMonitor::getStatusPcmcia( int showPopUp ) {
+bool CardMonitor::getStatusPcmcia(int showPopUp)
+{
 
     bool cardWas0 = cardInPcmcia0; // remember last state
@@ -191,10 +203,14 @@ bool CardMonitor::getStatusPcmcia( int showPopUp ) {
         streamIn = stream.read();
         list = QStringList::split( "\n", streamIn );
-        for( QStringList::Iterator line=list.begin(); line!=list.end(); line++ ) {
+	for (QStringList::Iterator line = list.begin(); line != list.end();
+	     line++) {
             if( (*line).startsWith( "Socket 0:" ) ){
                 if( (*line).startsWith( "Socket 0: empty" ) && cardInPcmcia0  ){
                     cardInPcmcia0 = FALSE;
-                } else if ( !(*line).startsWith( "Socket 0: empty" ) && !cardInPcmcia0 ){
-                    cardInPcmcia0Name = (*line).mid( ( (*line).find( ':' ) + 1 ), (*line).length() - 9 );
+		} else if (!(*line).startsWith("Socket 0: empty")
+			   && !cardInPcmcia0) {
+		    cardInPcmcia0Name =
+			(*line).mid(((*line).find(':') + 1),
+				    (*line).length() - 9);
                     cardInPcmcia0Name.stripWhiteSpace();
                     cardInPcmcia0 = TRUE;
@@ -202,11 +218,15 @@ bool CardMonitor::getStatusPcmcia( int showPopUp ) {
                     line++;
       int pos=(*line).find('\t' ) + 1;
-      cardInPcmcia0Type = (*line).mid( pos, (*line).find( "\t" , pos) - pos );
+		    cardInPcmcia0Type =
+			(*line).mid(pos, (*line).find("\t", pos) - pos);
                 }
             }  else if( (*line).startsWith( "Socket 1:" ) ){
                 if( (*line).startsWith( "Socket 1: empty" ) && cardInPcmcia1 ) {
                     cardInPcmcia1 = FALSE;
-                } else if ( !(*line).startsWith( "Socket 1: empty" ) && !cardInPcmcia1 ) {
-                    cardInPcmcia1Name = (*line).mid(((*line).find(':') + 1), (*line).length() - 9  );
+		} else if (!(*line).startsWith("Socket 1: empty")
+			   && !cardInPcmcia1) {
+		    cardInPcmcia1Name =
+			(*line).mid(((*line).find(':') + 1),
+				    (*line).length() - 9);
                     cardInPcmcia1Name.stripWhiteSpace();
                     cardInPcmcia1 = TRUE;
@@ -214,5 +234,6 @@ bool CardMonitor::getStatusPcmcia( int showPopUp ) {
                     line++;
       int pos=(*line).find('\t') + 1;
-      cardInPcmcia1Type = (*line).mid( pos, (*line).find("\t", pos) - pos );
+		    cardInPcmcia1Type =
+			(*line).mid(pos, (*line).find("\t", pos) - pos);
                 }
             }
@@ -220,5 +241,6 @@ bool CardMonitor::getStatusPcmcia( int showPopUp ) {
         f.close();
 
-        if( !showPopUp && (cardWas0 != cardInPcmcia0 || cardWas1 != cardInPcmcia1) ) {
+	if (!showPopUp
+	    && (cardWas0 != cardInPcmcia0 || cardWas1 != cardInPcmcia1)) {
             QString text = QString::null;
             QString what = QString::null;
@@ -254,9 +276,11 @@ bool CardMonitor::getStatusPcmcia( int showPopUp ) {
         cardInPcmcia1 = FALSE;
     }
-    return ( (cardWas0 == cardInPcmcia0 && cardWas1 == cardInPcmcia1 ) ? FALSE : TRUE );
+    return ((cardWas0 == cardInPcmcia0
+	     && cardWas1 == cardInPcmcia1) ? FALSE : TRUE);
 }
 
 
-bool CardMonitor::getStatusSd( int showPopUp ) {
+bool CardMonitor::getStatusSd(int showPopUp)
+{
 
     bool cardWas = cardInSd; // remember last state
@@ -271,5 +295,6 @@ bool CardMonitor::getStatusSd( int showPopUp ) {
             QString fs = me->mnt_fsname;
             //qDebug( fs );
-            if ( fs.left( 14 ) == "/dev/mmc/part1" || fs.left( 7 ) == "/dev/sd" || fs.left( 9 ) == "/dev/mmcd" ) {
+	    if (fs.left(14) == "/dev/mmc/part1" || fs.left(7) == "/dev/sd"
+		|| fs.left(9) == "/dev/mmcd") {
                 cardInSd = TRUE;
                 show();
@@ -286,8 +311,8 @@ bool CardMonitor::getStatusSd( int showPopUp ) {
         QString what = QString::null;
         if( cardInSd )  {
-            text += "SD Inserted";
+	    text += "New card: SD/MMC";
             what = "on";
         }  else {
-            text += "SD Removed";
+	    text += "Ejected: SD/MMC";
             what = "off";
         }
@@ -302,5 +327,6 @@ bool CardMonitor::getStatusSd( int showPopUp ) {
 }
 
-void CardMonitor::paintEvent( QPaintEvent * ) {
+void CardMonitor::paintEvent(QPaintEvent *)
+{
 
     QPainter p( this );