make obt translate keypress events to a valid utf8 character (note this makes openbox...

diff --git a/obt/keyboard.c b/obt/keyboard.c
index bd5d5df..82161e5 100644 (file)
--- a/obt/keyboard.c
+++ b/obt/keyboard.c
@@ -26,6 +26,8 @@ struct _ObtIC
 {
     guint ref;
     XIC xic;
 {
     guint ref;
     XIC xic;

+    Window client;
+    Window focus;

 };
 
 /* These masks are constants and the modifier keys are bound to them as
 };
 
 /* These masks are constants and the modifier keys are bound to them as


@@ -42,6 +44,7 @@ struct _ObtIC
 static void set_modkey_mask(guchar mask, KeySym sym);
 static void xim_init(void);
 void obt_keyboard_shutdown();
 static void set_modkey_mask(guchar mask, KeySym sym);
 static void xim_init(void);
 void obt_keyboard_shutdown();

+void obt_keyboard_context_renew(ObtIC *ic);

 
 static XModifierKeymap *modmap;
 static KeySym *keymap;
 
 static XModifierKeymap *modmap;
 static KeySym *keymap;


@@ -58,6 +61,7 @@ static gboolean started = FALSE;
 
 static XIM xim = NULL;
 static XIMStyle xim_style = 0;
 
 static XIM xim = NULL;
 static XIMStyle xim_style = 0;

+static GSList *xic_all = NULL;

 
 void obt_keyboard_reload(void)
 {
 
 void obt_keyboard_reload(void)
 {


@@ -124,6 +128,7 @@ void obt_keyboard_shutdown(void)
 
 void xim_init(void)
 {
 
 void xim_init(void)
 {

+    GSList *it;

     gchar *aname, *aclass;
 
     aname = g_strdup(g_get_prgname());
     gchar *aname, *aclass;
 
     aname = g_strdup(g_get_prgname());


@@ -169,6 +174,10 @@ void xim_init(void)
         }
     }
 
         }
     }
 

+    /* any existing contexts need to be recreated for the new input method */
+    for (it = xic_all; it; it = g_slist_next(it))
+        obt_keyboard_context_renew(it->data);
+

     g_free(aclass);
     g_free(aname);
 }
     g_free(aclass);
     g_free(aname);
 }


@@ -272,48 +281,113 @@ KeyCode* obt_keyboard_keysym_to_keycode(KeySym sym)
     return ret;
 }
 
     return ret;
 }
 

-gchar *obt_keyboard_keycode_to_string(guint keycode)
+gunichar obt_keyboard_keypress_to_unichar(ObtIC *ic, XKeyPressedEvent *ev)

 {
 {

+    gunichar unikey = 0;

     KeySym sym;
     KeySym sym;

+    Status status;
+    gchar *buf, fixbuf[4]; /* 4 is enough for a utf8 char */
+    gint len, bufsz;
+    gboolean got_string = FALSE;
+
+    if (!ic)
+        g_warning("Using obt_keyboard_keypress_to_unichar() without an "
+                  "Input Context.  No i18n support!");
+
+    if (ic && ic->xic) {
+        buf = fixbuf;
+        bufsz = sizeof(fixbuf);
+
+#ifdef X_HAVE_UTF8_STRING
+        len = Xutf8LookupString(ic->xic, ev, buf, bufsz, &sym, &status);
+#else
+        len = XmbLookupString(ic->xic, ev, buf, bufsz, &sym, &status);
+#endif
+
+        if (status == XBufferOverflow) {
+            buf = g_new(char, len);
+            bufsz = len;
+
+#ifdef X_HAVE_UTF8_STRING
+            len = Xutf8LookupString(ic->xic, ev, buf, bufsz, &sym, &status);
+#else
+            len = XmbLookupString(ic->xic, ev, buf, bufsz, &sym, &status);
+#endif
+        }

 
 

-    if ((sym = XKeycodeToKeysym(obt_display, keycode, 0)) != NoSymbol)
-        return g_locale_to_utf8(XKeysymToString(sym), -1, NULL, NULL, NULL);
-    return NULL;
-}
+        if ((status == XLookupChars || status == XLookupBoth)) {
+            if ((guchar)buf[0] >= 32) { /* not an ascii control character */
+#ifndef X_HAVE_UTF8_STRING
+                /* convert to utf8 */
+                gchar *buf2 = buf;
+                buf = g_locale_to_utf8(buf2, r, NULL, NULL, NULL);
+                g_free(buf2);
+#endif

 
 

-gunichar obt_keyboard_keycode_to_unichar(guint keycode)
-{
-    gunichar unikey = 0;
-    char *key;
-
-    if ((key = obt_keyboard_keycode_to_string(keycode)) != NULL &&
-        /* don't accept keys that aren't a single letter, like "space" */
-        key[1] == '\0')
-    {
-        unikey = g_utf8_get_char_validated(key, -1);
-        if (unikey == (gunichar)-1 || unikey == (gunichar)-2 || unikey == 0)
-            unikey = 0;
+                got_string = TRUE;
+            }
+        }
+        else
+            g_message("Bad keycode lookup. Keysym 0x%x Status: %s\n",
+                      (guint) sym,
+                      (status == XBufferOverflow ? "BufferOverflow" :
+                       status == XLookupNone ? "XLookupNone" :
+                       status == XLookupKeySym ? "XLookupKeySym" :
+                       "Unknown status"));
+    }
+    else {
+        buf = fixbuf;
+        bufsz = sizeof(fixbuf);
+        len = XLookupString(ev, buf, bufsz, &sym, NULL);
+        if ((guchar)buf[0] >= 32) /* not an ascii control character */
+            got_string = TRUE;
+    }
+
+    if (got_string) {
+        gunichar u = g_utf8_get_char_validated(buf, len);
+        if (u && u != (gunichar)-1 && u != (gunichar)-2)
+            unikey = u;

     }
     }

-    g_free(key);
+
+    if (buf != fixbuf) g_free(buf);
+

     return unikey;
 }
 
     return unikey;
 }
 

-ObtIC* obt_keyboard_context_new(Window w)
+void obt_keyboard_context_renew(ObtIC *ic)

 {
 {

-    ObtIC *ic = NULL;
-
-    if (w != None) {
-        ic = g_new(ObtIC, 1);
-        ic->ref = 1;
+    if (ic->xic) {
+        XDestroyIC(ic->xic);

         ic->xic = NULL;
         ic->xic = NULL;

+    }

 
 

-        if (xim)
-            ic->xic = XCreateIC(xim,
-                                XNInputStyle, xim_style,
-                                XNClientWindow, w,
-                                XNFocusWindow, w,
-                                NULL);
+    if (xim) {
+        ic->xic = XCreateIC(xim,
+                            XNInputStyle, xim_style,
+                            XNClientWindow, ic->client,
+                            XNFocusWindow, ic->focus,
+                            NULL);
+        if (!ic->xic)
+            g_message("Error creating Input Context for window 0x%x 0x%x\n",
+                      (guint)ic->client, (guint)ic->focus);

     }
     }

+}
+
+ObtIC* obt_keyboard_context_new(Window client, Window focus)
+{
+    ObtIC *ic;
+
+    g_return_val_if_fail(client != None && focus != None, NULL);
+
+    ic = g_new(ObtIC, 1);
+    ic->ref = 1;
+    ic->client = client;
+    ic->focus = focus;
+    ic->xic = NULL;
+
+    obt_keyboard_context_renew(ic);
+    xic_all = g_slist_prepend(xic_all, ic);
+

     return ic;
 }
 
     return ic;
 }
 


@@ -325,6 +399,7 @@ void obt_keyboard_context_ref(ObtIC *ic)
 void obt_keyboard_context_unref(ObtIC *ic)
 {
     if (--ic->ref < 1) {
 void obt_keyboard_context_unref(ObtIC *ic)
 {
     if (--ic->ref < 1) {

+        xic_all = g_slist_remove(xic_all, ic);

         XDestroyIC(ic->xic);
         g_free(ic);
     }
         XDestroyIC(ic->xic);
         g_free(ic);
     }

diff --git a/obt/keyboard.h b/obt/keyboard.h
index d166faa..902f95b 100644 (file)
--- a/obt/keyboard.h
+++ b/obt/keyboard.h
@@ -59,14 +59,14 @@ guint obt_keyboard_modkey_to_modmask(ObtModkeysKey key);
 /*! Convert a KeySym to all the KeyCodes which generate it. */
 KeyCode* obt_keyboard_keysym_to_keycode(KeySym sym);
 
 /*! Convert a KeySym to all the KeyCodes which generate it. */
 KeyCode* obt_keyboard_keysym_to_keycode(KeySym sym);
 

-/*! Give the string form of a KeyCode */
-gchar *obt_keyboard_keycode_to_string(guint keycode);
+/*! Translate a KeyPress event to the unicode character it represents */
+gunichar obt_keyboard_keypress_to_unichar(ObtIC *ic, XKeyPressedEvent *ev);

 
 

-/*! Translate a KeyCode to the unicode character it represents */
-gunichar obt_keyboard_keycode_to_unichar(guint keycode);
-
-/*! Create an input context for a window */
-ObtIC* obt_keyboard_context_new(Window w);
+/*! Create an input context for a window.
+    @client The top-level client window for the input context.
+    @focus The subwindow within the client for the input context.
+*/
+ObtIC* obt_keyboard_context_new(Window client, Window focus);

 
 void obt_keyboard_context_ref(ObtIC *ic);
 void obt_keyboard_context_unref(ObtIC *ic);
 
 void obt_keyboard_context_ref(ObtIC *ic);
 void obt_keyboard_context_unref(ObtIC *ic);

diff --git a/openbox/prompt.c b/openbox/prompt.c
index 7353615..9fd5677 100644 (file)
--- a/openbox/prompt.c
+++ b/openbox/prompt.c
@@ -166,6 +166,8 @@ ObPrompt* prompt_new(const gchar *msg, const gchar *title,
                                        CopyFromParent,
                                        CWOverrideRedirect,
                                        &attrib);
                                        CopyFromParent,
                                        CWOverrideRedirect,
                                        &attrib);

+    self->ic = obt_keyboard_context_new(self->super.window,
+                                        self->super.window);

 
     /* make it a dialog type window */
     OBT_PROP_SET32(self->super.window, NET_WM_WINDOW_TYPE, ATOM,
 
     /* make it a dialog type window */
     OBT_PROP_SET32(self->super.window, NET_WM_WINDOW_TYPE, ATOM,


@@ -239,6 +241,8 @@ void prompt_unref(ObPrompt *self)
 
         prompt_list = g_list_remove(prompt_list, self);
 
 
         prompt_list = g_list_remove(prompt_list, self);
 

+        obt_keyboard_context_unref(self->ic);
+

         for (i = 0; i < self->n_buttons; ++i) {
             window_remove(self->button[i].window);
             XDestroyWindow(obt_display, self->button[i].window);
         for (i = 0; i < self->n_buttons; ++i) {
             window_remove(self->button[i].window);
             XDestroyWindow(obt_display, self->button[i].window);

diff --git a/openbox/prompt.h b/openbox/prompt.h
index 3c46c30..1358574 100644 (file)
--- a/openbox/prompt.h
+++ b/openbox/prompt.h
@@ -22,6 +22,7 @@
 #include "window.h"
 #include "geom.h"
 #include "obrender/render.h"
 #include "window.h"
 #include "geom.h"
 #include "obrender/render.h"

+#include "obt/keyboard.h"

 #include <glib.h>
 #include <X11/Xlib.h>
 
 #include <glib.h>
 #include <X11/Xlib.h>
 


@@ -47,6 +48,7 @@ struct _ObPrompt
     ObInternalWindow super;
     gint ref;
 
     ObInternalWindow super;
     gint ref;
 

+    ObtIC *ic;

     guint event_mask;
 
     /* keep a copy of this because we re-render things that may need it
     guint event_mask;
 
     /* keep a copy of this because we re-render things that may need it