author | zecke <zecke> | 2004-09-18 19:43:37 (UTC) |
---|---|---|
committer | zecke <zecke> | 2004-09-18 19:43:37 (UTC) |
commit | af0d773a9991dec27d25c2ad6859ee2abe23c73a (patch) (unidiff) | |
tree | db35921190287e0c0b19c6207e8d87f679c513ee /core/applets/cardmon/cardmon.cpp | |
parent | 3f56f22d205a7f2a60262aa026542e9dc844810f (diff) | |
download | opie-af0d773a9991dec27d25c2ad6859ee2abe23c73a.zip opie-af0d773a9991dec27d25c2ad6859ee2abe23c73a.tar.gz opie-af0d773a9991dec27d25c2ad6859ee2abe23c73a.tar.bz2 |
-Killl Magic Numbers and introduce an enum
-Split the Command Line on Creation, this way we do not need to
split the command afterwards
Diffstat (limited to 'core/applets/cardmon/cardmon.cpp') (more/less context) (ignore whitespace changes)
-rw-r--r-- | core/applets/cardmon/cardmon.cpp | 57 |
1 files changed, 31 insertions, 26 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 | |||
@@ -109,5 +109,6 @@ void CardMonitor::popupTimeout() { | |||
109 | void CardMonitor::mousePressEvent( QMouseEvent * ) { | 109 | void CardMonitor::mousePressEvent( QMouseEvent * ) { |
110 | QPopupMenu * menu = new QPopupMenu( this ); | 110 | QPopupMenu * menu = new QPopupMenu( this ); |
111 | QString cmd; | 111 | QStringList cmd; |
112 | bool execute = true; | ||
112 | 113 | ||
113 | if ( cardInSd ) { | 114 | if ( cardInSd ) { |
@@ -136,13 +137,18 @@ void CardMonitor::mousePressEvent( QMouseEvent * ) { | |||
136 | 137 | ||
137 | if ( opt == 1 ) { | 138 | if ( opt == 1 ) { |
138 | m_commandOrig = 1; | 139 | m_commandOrig = PCMCIA_Socket1; |
139 | execCommand("/sbin/cardctl eject 0"); | 140 | cmd << "/sbin/cardctl" << "eject" << "0"; |
140 | } else if ( opt == 0 ) { | 141 | } else if ( opt == 0 ) { |
141 | m_commandOrig = 2; | 142 | m_commandOrig = MMC_Socket; |
142 | execCommand( QString("umount %1").arg(cardSdName)); | 143 | cmd << "umount" << cardSdName; |
143 | } else if ( opt == 2 ) { | 144 | } else if ( opt == 2 ) { |
144 | m_commandOrig = 3; | 145 | m_commandOrig = PCMCIA_Socket2; |
145 | execCommand( "/sbin/cardctl eject 1" ); | 146 | cmd << "/sbin/cardctl" << "eject" << "1"; |
146 | } | 147 | }else |
148 | execute = false; | ||
149 | |||
150 | if ( execute ) | ||
151 | execCommand( cmd ); | ||
152 | |||
147 | delete menu; | 153 | delete menu; |
148 | } | 154 | } |
@@ -334,5 +340,5 @@ int CardMonitor::position() { | |||
334 | } | 340 | } |
335 | 341 | ||
336 | void CardMonitor::execCommand( const QString &command ) { | 342 | void CardMonitor::execCommand( const QStringList &strList ) { |
337 | delete m_process; | 343 | delete m_process; |
338 | m_process = 0; | 344 | m_process = 0; |
@@ -340,12 +346,12 @@ void CardMonitor::execCommand( const QString &command ) { | |||
340 | if ( m_process == 0 ) { | 346 | if ( m_process == 0 ) { |
341 | m_process = new OProcess(); | 347 | m_process = new OProcess(); |
342 | QStringList strList = QStringList::split( " ", command ); | ||
343 | 348 | ||
344 | for ( QStringList::Iterator it = strList.begin(); it != strList.end(); ++it ) { | ||
345 | *m_process << *it; | ||
346 | } | ||
347 | 349 | ||
348 | connect(m_process, SIGNAL( processExited(Opie::Core::OProcess*) ), | 350 | for ( QStringList::ConstIterator it = strList.begin(); it != strList.end(); ++it ) { |
349 | this, SLOT( slotExited(Opie::Core::OProcess* ) ) ); | 351 | *m_process << *it; |
352 | } | ||
353 | |||
354 | connect(m_process, SIGNAL(processExited(Opie::Core::OProcess*)), | ||
355 | this, SLOT( slotExited(Opie::Core::OProcess*))); | ||
350 | 356 | ||
351 | if(!m_process->start(OProcess::NotifyOnExit, OProcess::AllOutput ) ) { | 357 | if(!m_process->start(OProcess::NotifyOnExit, OProcess::AllOutput ) ) { |
@@ -359,15 +365,14 @@ void CardMonitor::slotExited( OProcess* ) { | |||
359 | 365 | ||
360 | if( m_process->normalExit() ) { | 366 | if( m_process->normalExit() ) { |
361 | int ret = m_process->exitStatus(); | 367 | int ret = m_process->exitStatus(); |
362 | if( ret != 0 ) { | 368 | if( ret != 0 ) { |
363 | if ( m_commandOrig == 1 ) { | 369 | if ( m_commandOrig == PCMCIA_Socket1 || |
364 | popUp( tr( "CF/PCMCIA card eject failed!" ) ); | 370 | m_commandOrig == PCMCIA_Socket2 ) { |
365 | } else if ( m_commandOrig == 2 ) { | 371 | popUp( tr( "CF/PCMCIA card eject failed!" ) ); |
366 | popUp( tr( "SD/MMC card eject failed!" ) ); | 372 | } else if ( m_commandOrig == MMC_Socket ) { |
367 | } else if ( m_commandOrig == 3 ) { | 373 | popUp( tr( "SD/MMC card eject failed!" ) ); |
368 | popUp( tr( "CF/PCMCIA card eject failed!" ) ); | 374 | } |
369 | } | 375 | } |
370 | } | 376 | } |
371 | } | ||
372 | } | 377 | } |
373 | 378 | ||