all repos — openbox @ c4ba6e605a07c177d773323d138f0caa0f334c97

openbox fork - make it a bit more like ryudo 

assume a valid utf8 char is at the shortcut position. the menu parser makes sure of this
Dana Jansens danakj@orodu.net
commit

c4ba6e605a07c177d773323d138f0caa0f334c97

parent

baa9ad14803fd3019f8da364439dafe9c0e1a9f7

1 files changed, 10 insertions(+), 12 deletions(-)

jump to
M render/font.c → render/font.c 
@@ -252,19 +252,17 @@ 
     if (t->shortcut) {
         const gchar *c = t->string + t->shortcut_pos;
 
-        if (g_utf8_validate(c, -1, NULL)) {
-            t->font->shortcut_underline->start_index = t->shortcut_pos;
-            t->font->shortcut_underline->end_index = t->shortcut_pos +
-                (g_utf8_next_char(c) - c);
+        t->font->shortcut_underline->start_index = t->shortcut_pos;
+        t->font->shortcut_underline->end_index = t->shortcut_pos +
+            (g_utf8_next_char(c) - c);
 
-            /* the attributes are owned by the layout.
-               re-add the attributes to the layout after changing the
-               start and end index */
-            attrlist = pango_layout_get_attributes(t->font->layout);
-            pango_attr_list_ref(attrlist);
-            pango_layout_set_attributes(t->font->layout, attrlist);
-            pango_attr_list_unref(attrlist);
-        }
+        /* the attributes are owned by the layout.
+           re-add the attributes to the layout after changing the
+           start and end index */
+        attrlist = pango_layout_get_attributes(t->font->layout);
+        pango_attr_list_ref(attrlist);
+        pango_layout_set_attributes(t->font->layout, attrlist);
+        pango_attr_list_unref(attrlist);
     }
 
     /* layout_line() uses y to specify the baseline