trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: November 2011

patch fails with qt 3.3.8d

From: Robert Xu <robxu9@...>
Date: Mon, 14 Nov 2011 15:35:42 -0500
Hi all,

Usually I get the already applied message for patches, however I seem
to have something different now:
This patch fails completely.

Can anyone make heads or tails out of it, or should this patch be
significant enough to keep or throw away?

Patch #28 (fix-key-release-event-with-imm.diff):
--- src/kernel/qapplication_x11.cpp
+++ src/kernel/qapplication_x11.cpp
@@ -3294,11 +3294,7 @@
     // filtering opportunity first to ensure all input methods work
     // properly regardless of application design.

-#ifndef QT_NO_IM_EXTENSIONS
     if( keywidget && keywidget->isEnabled() &&
keywidget->isInputMethodEnabled() ) {
-#else
-    if( keywidget && keywidget->isEnabled() ) {
-#endif
 	if( ( event->type==XKeyPress || event->type==XKeyRelease ) &&
 	    sm_blockUserInput ) // block user interaction during session management
 	    return TRUE;
@@ -5220,11 +5216,12 @@
     } else {
 	key = (int)(long)keyDict->find( keycode );
 	if ( key )
-	    if( !willRepeat ) // Take out key of dictionary only if this call.
+	    if( !willRepeat && statefulTranslation ) // Take out key of
dictionary only if this call.
 		keyDict->take( keycode );
 	long s = (long)textDict->find( keycode );
 	if ( s ) {
-	    textDict->take( keycode );
+	    if( statefulTranslation )	
+	        textDict->take( keycode );
 	    ascii = (char)(s-256);
 	}
     }
--- src/kernel/qwidget_x11.cpp
+++ src/kernel/qwidget_x11.cpp
@@ -2699,11 +2699,10 @@
 {
     QInputContext *qic = 0;

-#if !defined(QT_NO_IM_EXTENSIONS)
     if ( isInputMethodEnabled() ) {
+#if !defined(QT_NO_IM_EXTENSIONS)
 	qic = icHolderWidget()->ic;
 #else
-    {
 	// icHolderWidget is always topLevelWidget
 	QTLWExtra *topdata = icHolderWidget()->topData();
 	qic = (QInputContext *)topdata->xic;
@@ -2754,10 +2753,8 @@
 */
 void QWidget::createInputContext()
 {
-#if !defined(QT_NO_IM_EXTENSIONS)
     if( !isInputMethodEnabled() || QApplication::closingDown() )
 	return;
-#endif

     QWidget *icWidget = icHolderWidget();
 #ifndef QT_NO_IM

-- 
later daze. :: Robert Xu :: rxu.lincomlinux.org :: protocol.by/rxu