Commit: 1ac01a0d238cc1124db6349afca8faa3cf394228 - uim (git)

A multilingual input method framework

Commit MetaInfo

• Revision: 1ac01a0d238cc1124db6349afca8faa3cf394228 (tree)
• Zeit: 2005-09-01 01:54:16
• Autor: ekato <ekato@ff9a...>
• Commiter: ekato

Log Message

* xim/ : Port changes in r1366 from trunk.

Ändern Zusammenfassung

• modified: xim/ximic.cpp
• modified: xim/ximserver.cpp
• modified: xim/ximserver.h

Diff

--- a/xim/ximic.cpp
+++ b/xim/ximic.cpp
@@ -407,6 +407,9 @@ void XimIC::setFocus()
     }
 }
 
+// Note that the sequence of XIM_SET_IC_FOCUS and XIM_UNSET_FOCUS
+// events is not consistent.  Be careful about hiding caret state and
+// candidate window.
 void XimIC::unsetFocus()
 {
     if (!mIsActive)
@@ -415,14 +418,6 @@ void XimIC::unsetFocus()
     mIsActive = false;
     nrActiveIC--;
     m_kkContext->focusOut();
-    // Since the sequence of XIM_SET_IC_FOCUS and XIM_UNSET_FOCUS
-    // events is not consistent, unsetting focus of candidate window
-    // is now handled in XimIC::setFocus() and focus_in message from
-    // helper application.
-    //
-    // if (mConvdisp && m_kkContext->hasActiveCandwin()) {
-    //     mConvdisp->unset_focus();
-    // }
 }
 
 void XimIC::OnKeyEvent(keyEventX e)

--- a/xim/ximserver.cpp
+++ b/xim/ximserver.cpp
@@ -305,7 +305,7 @@ InputContext::~InputContext()
 	mFocusedContext = NULL;
 
     if (mConvdisp)
-	mConvdisp->set_pe(0);
+	mConvdisp->set_pe(NULL);
 
     delete m_pe;
     uim_release_context(mUc);
@@ -451,6 +451,13 @@ void
 InputContext::focusOut()
 {
     uim_helper_client_focus_out(mUc);
+    if (mFocusedContext == this) {
+	Canddisp *disp = canddisp_singleton();
+	if (isCaretStateShown())
+	    disp->hide_caret_state();
+	if (hasActiveCandwin())
+	    disp->hide();
+    }
 }
 
 XimServer *
@@ -471,7 +478,7 @@ InputContext::commit_cb(void *ptr, const char *str)
     InputContext *ic = (InputContext *)ptr;
     XimIC *xic = ic->get_ic();
 
-    clear_cb(ptr);
+    ic->clear_pe_stat();
     ic->update_preedit();
     xic->commit_string(str);
 }
@@ -479,7 +486,7 @@ InputContext::commit_cb(void *ptr, const char *str)
 void InputContext::clear_cb(void *ptr)
 {
     InputContext *ic = (InputContext *)ptr;
-    ic->clear_preedit();
+    ic->clear_pe_stat();
 }
 
 void InputContext::pushback_cb(void *ptr, int attr, const char *str)
@@ -534,11 +541,18 @@ void InputContext::update_prop_label_cb(void *ptr, const char *str)
       ic->update_prop_label(str);
 }
 
-void InputContext::clear_preedit()
+void InputContext::clear_pe_stat()
 {
     m_pe->clear();
 }
 
+void InputContext::clear_preedit()
+{
+    clear_pe_stat();
+    if (mConvdisp)
+	mConvdisp->clear_preedit();
+}
+
 uString InputContext::get_preedit_string()
 {
     uString str;
@@ -614,11 +628,10 @@ bool InputContext::hasActiveCandwin()
     return mCandwinActive;
 }
 
+// reset
 void InputContext::clear()
 {
     clear_preedit();
-    if (mConvdisp)
-	mConvdisp->clear_preedit();
     candidate_deactivate();
     uim_reset_context(mUc);
 }

--- a/xim/ximserver.h
+++ b/xim/ximserver.h
@@ -193,15 +193,15 @@ public:
     static void update_prop_label_cb(void *ptr, const char *str);
     static InputContext *focusedContext();
     static void deletefocusedContext();
-protected:
+private:
     void commit_string(char *);
+    void clear_pe_stat();
+
     XimIC *mXic;
     XimServer *mServer;
     pe_stat *m_pe;
     Convdisp *mConvdisp;
     uim_context mUc;
-private:
-    static InputContext *mFocusedContext;
     bool mCandwinActive;
     int mDisplayLimit;
     int mNumPage;
@@ -211,6 +211,8 @@ private:
     char *mEngineName;
     char *mLocaleName;
     bool mCaretStateShown;
+private:
+    static InputContext *mFocusedContext;
 };
 
 class Locale {