Don't lock up with Qt2.3.8

1 files changed, 2 insertions, 2 deletions

diff --git a/library/qpeapplication.cpp b/library/qpeapplication.cpp
index 5d05ed5..3370264 100644
--- a/library/qpeapplication.cpp
+++ b/library/qpeapplication.cpp
@@ -949,97 +949,97 @@ bool QPEApplication::qwsEventFilter( QWSEvent * e )
 				}
 			}
 		}
 		else if ( ke->simpleData.keycode >= Qt::Key_F1 && ke->simpleData.keycode <= Qt::Key_F29 ) {
 			// this should be if ( ODevice::inst ( )-> buttonForKeycode ( ... ))
 			// but we cannot access libopie function within libqpe :(
 
 			QWidget * active = activeWindow ( );
 			if ( active && ((int) active-> winId ( ) == ke-> simpleData.window )) {
 				if ( d-> kbgrabbed ) { // we grabbed the keyboard
 					QChar ch ( ke-> simpleData.unicode );
 					QKeyEvent qke ( ke-> simpleData. is_press ? QEvent::KeyPress : QEvent::KeyRelease,
 					                ke-> simpleData.keycode,
 					                ch. latin1 ( ),
 					                ke-> simpleData.modifiers,
 					                QString ( ch ),
 					                ke-> simpleData.is_auto_repeat, 1 );
 
 					QObject *which = QWidget::keyboardGrabber ( );
 					if ( !which )
 						which = QApplication::focusWidget ( );
 					if ( !which )
 						which = QApplication::activeWindow ( );
 					if ( !which )
 						which = qApp;
 
 					QApplication::sendEvent ( which, &qke );
 				}
 				else { // we didn't grab the keyboard, so send the event to the launcher
 					QCopEnvelope e ( "QPE/Launcher", "deviceButton(int,int,int)" );
 					e << int( ke-> simpleData.keycode ) << int( ke-> simpleData. is_press ) << int( ke-> simpleData.is_auto_repeat );
 				}
 			}
 			return true;
 		}
 	}
 	if ( e->type == QWSEvent::Focus ) {
 		QWSFocusEvent * fe = ( QWSFocusEvent* ) e;
 		if ( !fe->simpleData.get_focus ) {
 			QWidget * active = activeWindow();
 			while ( active && active->isPopup() ) {
 				active->close();
 				active = activeWindow();
 			}
 		}
 		else {
 			// make sure our modal widget is ALWAYS on top
 			QWidget *topm = activeModalWidget();
-			if ( topm ) {
+			if ( topm && static_cast<int>( topm->winId() ) != fe->simpleData.window) {
 				topm->raise();
 			}
 		}
 		if ( fe->simpleData.get_focus && inputMethodDict ) {
 			InputMethodHint m = inputMethodHint( QWidget::find( e->window() ) );
 			if ( m == AlwaysOff )
 				Global::hideInputMethod();
 			if ( m == AlwaysOn )
 				Global::showInputMethod();
 		}
 	}
 
 
 	return QApplication::qwsEventFilter( e );
 }
 #endif
 
 /*!
   Destroys the QPEApplication.
 */
 QPEApplication::~QPEApplication()
 {
 	ungrabKeyboard();
 #if defined(Q_WS_QWS) && !defined(QT_NO_COP)
 	// Need to delete QCopChannels early, since the display will
 	// be gone by the time we get to ~QObject().
 	delete sysChannel;
 	delete pidChannel;
 #endif
 
 	delete d;
 }
 
 /*!
   Returns <tt>$OPIEDIR/</tt>.
 */
 QString QPEApplication::qpeDir()
 {
 	const char * base = getenv( "OPIEDIR" );
 	if ( base )
 		return QString( base ) + "/";
 
 	return QString( "../" );
 }
 
 /*!
   Returns the user's current Document directory. There is a trailing "/".
   .. well, it does now,, and there's no trailing '/'
@@ -1420,97 +1420,97 @@ bool QPEApplication::raiseAppropriateWindow()
 	    r = TRUE;
 	else if (d->preloaded) {
 	    // We are preloaded and not visible.. pretend we just started..
 #ifndef QT_NO_COP
 	    QCopEnvelope e("QPE/System", "fastAppShowing(QString)");
 	    e << d->appName;
 #endif
 	}
 
 	d->show_mx(top,d->nomaximize, d->appName);
 	top->raise();
     }
 
     QWidget *topm = activeModalWidget();
 
     // 2. Raise any parentless widgets (except top and topm, as they
     //     are raised before and after this loop).  Order from most
     //     recently raised as deepest to least recently as top, so
     //     that repeated calls cycle through widgets.
     QWidgetList *list = topLevelWidgets();
     if ( list ) {
 	bool foundlast = FALSE;
 	QWidget* topsub = 0;
 	if ( d->lastraised ) {
 	    for (QWidget* w = list->first(); w; w = list->next()) {
 		if ( !w->parentWidget() && w != topm && w->isVisible() && !w->isDesktop() ) {
 		    if ( w == d->lastraised )
 			foundlast = TRUE;
 		    if ( foundlast ) {
 			w->raise();
 			topsub = w;
 		    }
 		}
 	    }
 	}
 	for (QWidget* w = list->first(); w; w = list->next()) {
 	    if ( !w->parentWidget() && w != topm && w->isVisible() && !w->isDesktop() ) {
 		if ( w == d->lastraised )
 		    break;
 		w->raise();
 		topsub = w;
 	    }
 	}
 	d->lastraised = topsub;
 	delete list;
     }
 
     // 3. Raise the active modal widget.
-    if ( topm && topm != top ) {
+    if ( topm ) {
 	topm->show();
 	topm->raise();
 	// If we haven't already handled the fastAppShowing message
 	if (!top && d->preloaded) {
 #ifndef QT_NO_COP
 	    QCopEnvelope e("QPE/System", "fastAppShowing(QString)");
 	    e << d->appName;
 #endif
 	}
 	r = FALSE;
     }
 
     return r;
 }
 
 
 void QPEApplication::pidMessage( const QCString& msg, const QByteArray& data)
 {
 #ifdef Q_WS_QWS
 
 	if ( msg == "quit()" ) {
 		tryQuit();
 	}
 	else if ( msg == "quitIfInvisible()" ) {
 		if ( d->qpe_main_widget && !d->qpe_main_widget->isVisible() )
 			quit();
 	}
 	else if ( msg == "close()" ) {
 		hideOrQuit();
 	}
 	else if ( msg == "disablePreload()" ) {
 		d->preloaded = FALSE;
 		d->keep_running = TRUE;
 		/* so that quit will quit */
 	}
 	else if ( msg == "enablePreload()" ) {
 		if (d->qpe_main_widget)
 			d->preloaded = TRUE;
 		d->keep_running = TRUE;
 		/* so next quit won't quit */
 	}
 	else if ( msg == "raise()" ) {
 		d->keep_running = TRUE;
 		d->notbusysent = FALSE;
 		raiseAppropriateWindow();
 		// Tell the system we're still chugging along...
 		QCopEnvelope e("QPE/System", "appRaised(QString)");
 		e << d->appName;