Enlightenment CVS committal Author : raster Project : e17 Module : apps/e
Dir : e17/apps/e/src/bin Modified Files: e_int_config_keybindings.c Log Message: fix keysym -> keyname && stringshare add missing =================================================================== RCS file: /cvs/e/e17/apps/e/src/bin/e_int_config_keybindings.c,v retrieving revision 1.17 retrieving revision 1.18 diff -u -3 -r1.17 -r1.18 --- e_int_config_keybindings.c 1 Apr 2006 09:24:56 -0000 1.17 +++ e_int_config_keybindings.c 17 Apr 2006 12:24:33 -0000 1.18 @@ -1371,7 +1371,7 @@ return; bk->context = E_BINDING_CONTEXT_ANY; - bk->key = strdup(""); + bk->key = evas_stringshare_add(""); bk->modifiers = E_BINDING_MODIFIER_NONE; bk->action = actions_predefined_names[cfdata->cur_eckb->acn].action_cmd == NULL ? evas_stringshare_add("") : @@ -1482,90 +1482,90 @@ static int _e_keybinding_cb_shortcut_key_down(void *data, int type, void *event) { - E_Config_Binding_Key *bk; - E_Config_Dialog_Data *cfdata = data; - Ecore_X_Event_Key_Down *ev = event; - - if (ev->win != cfdata->locals.keybind_win) return 1; - - if (!strcmp(ev->keysymbol, "Escape") && - !(ev->modifiers & ECORE_X_MODIFIER_SHIFT) && - !(ev->modifiers & ECORE_X_MODIFIER_CTRL) && - !(ev->modifiers & ECORE_X_MODIFIER_ALT) && - !(ev->modifiers & ECORE_X_MODIFIER_WIN)) - { - _e_keybinding_keybind_shortcut_wnd_hide(cfdata); - } - else - { - - if (!strcmp(ev->keysymbol, "Control_L") || !strcmp(ev->keysymbol, "Control_R") || - !strcmp(ev->keysymbol, "Shift_L") || !strcmp(ev->keysymbol, "Shift_R") || - !strcmp(ev->keysymbol, "Alt_L") || !strcmp(ev->keysymbol, "Alt_R") || - !strcmp(ev->keysymbol, "Super_L") || !strcmp(ev->keysymbol, "Super_R")) - ; - else - { - if (cfdata && cfdata->cur_eckb && cfdata->cur_eckb_kb_sel >= 0 && - cfdata->cur_eckb->bk_list) - { - Evas_List *l, *l2; - - E_Config_KeyBind *eckb; - E_Config_Binding_Key *bk_tmp; - - int found = 0; - int mod = E_BINDING_MODIFIER_NONE; - - if (ev->modifiers & ECORE_X_MODIFIER_SHIFT) - mod |= E_BINDING_MODIFIER_SHIFT; - if (ev->modifiers & ECORE_X_MODIFIER_CTRL) - mod |= E_BINDING_MODIFIER_CTRL; - if (ev->modifiers & ECORE_X_MODIFIER_ALT) - mod |= E_BINDING_MODIFIER_ALT; - if (ev->modifiers & ECORE_X_MODIFIER_WIN) - mod |= E_BINDING_MODIFIER_WIN; - - for (l = cfdata->key_bindings; l && !found; l = l->next) - { - eckb = l->data; - for (l2 = eckb->bk_list; l2 && !found; l2 = l2->next) + E_Config_Binding_Key *bk; + E_Config_Dialog_Data *cfdata = data; + Ecore_X_Event_Key_Down *ev = event; + + if (ev->win != cfdata->locals.keybind_win) return 1; + + if (!strcmp(ev->keyname, "Escape") && + !(ev->modifiers & ECORE_X_MODIFIER_SHIFT) && + !(ev->modifiers & ECORE_X_MODIFIER_CTRL) && + !(ev->modifiers & ECORE_X_MODIFIER_ALT) && + !(ev->modifiers & ECORE_X_MODIFIER_WIN)) + { + _e_keybinding_keybind_shortcut_wnd_hide(cfdata); + } + else + { + printf("'%s' '%s' '%s'\n", ev->keyname, ev->keysymbol, ev->key_compose); + if (!strcmp(ev->keyname, "Control_L") || !strcmp(ev->keyname, "Control_R") || + !strcmp(ev->keyname, "Shift_L") || !strcmp(ev->keyname, "Shift_R") || + !strcmp(ev->keyname, "Alt_L") || !strcmp(ev->keyname, "Alt_R") || + !strcmp(ev->keyname, "Super_L") || !strcmp(ev->keyname, "Super_R")) + ; + else + { + if (cfdata && cfdata->cur_eckb && cfdata->cur_eckb_kb_sel >= 0 && + cfdata->cur_eckb->bk_list) + { + Evas_List *l, *l2; + + E_Config_KeyBind *eckb; + E_Config_Binding_Key *bk_tmp; + + int found = 0; + int mod = E_BINDING_MODIFIER_NONE; + + if (ev->modifiers & ECORE_X_MODIFIER_SHIFT) + mod |= E_BINDING_MODIFIER_SHIFT; + if (ev->modifiers & ECORE_X_MODIFIER_CTRL) + mod |= E_BINDING_MODIFIER_CTRL; + if (ev->modifiers & ECORE_X_MODIFIER_ALT) + mod |= E_BINDING_MODIFIER_ALT; + if (ev->modifiers & ECORE_X_MODIFIER_WIN) + mod |= E_BINDING_MODIFIER_WIN; + + for (l = cfdata->key_bindings; l && !found; l = l->next) { - bk_tmp = l2->data; - - if (bk_tmp->modifiers == mod && !strcmp(ev->keysymbol, bk_tmp->key)) - found = 1; + eckb = l->data; + for (l2 = eckb->bk_list; l2 && !found; l2 = l2->next) + { + bk_tmp = l2->data; + + if (bk_tmp->modifiers == mod && !strcmp(ev->keyname, bk_tmp->key)) + found = 1; + } } - } - - if (!found) - { - bk = evas_list_nth(cfdata->cur_eckb->bk_list, cfdata->cur_eckb_kb_sel); - bk->modifiers = E_BINDING_MODIFIER_NONE; - - bk->modifiers = mod; - - if (bk->key) - evas_stringshare_del(bk->key); - bk->key = evas_stringshare_add(ev->keysymbol); - - _e_keybinding_update_binding_list(cfdata); - } - else - { - e_util_dialog_show(_("Binding Key Error"), - _("The binding key sequence, that you choose," - " is already used.<br>Please choose another binding key" - " sequence.")); - } - _e_keybinding_keybind_shortcut_wnd_hide(cfdata); - } - } - } - return 1; + + if (!found) + { + bk = evas_list_nth(cfdata->cur_eckb->bk_list, cfdata->cur_eckb_kb_sel); + bk->modifiers = E_BINDING_MODIFIER_NONE; + + bk->modifiers = mod; + + if (bk->key) + evas_stringshare_del(bk->key); + bk->key = evas_stringshare_add(ev->keyname); + + _e_keybinding_update_binding_list(cfdata); + } + else + { + e_util_dialog_show(_("Binding Key Error"), + _("The binding key sequence, that you choose," + " is already used.<br>Please choose another binding key" + " sequence.")); + } + _e_keybinding_keybind_shortcut_wnd_hide(cfdata); + } + } + } + return 1; } static int _e_keybinding_cb_mouse_handler_dumb(void *data, int type, void *event) { - return 1; + return 1; } ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642 _______________________________________________ enlightenment-cvs mailing list enlightenment-cvs@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs