Dear Vincent, Hello~

Right side control and alt key does not work.
It does not work on the Ubuntu also.
So I have removed for control and alt. Please check the attached file.

Sincerely,
Shinwoo Kim.

2011/11/21 Vincent Torri <[email protected]>

>
>
> On Mon, 21 Nov 2011, Kim Shinwoo wrote:
>
> > Dear Mr. Vincent, Hello!
> >
> > I have verified. It works properly.
> > When I press the left (or right) shift key, it gives Shift_L (or
> Shift_R).
> > When I release the left (or right) shift key, it gives Shift_L (or
> > Shift_R). :)
>
> check control key too
>
> Vincent
>
> >
> > Sincerely,
> > Shinwoo Kim.
> >
> >
> >
> > 2011/11/20 Vincent Torri <[email protected]>
> >
> >>
> >>
> >> On Thu, 17 Nov 2011, Kim Shinwoo wrote:
> >>
> >>> Dear All, Hello~
> >>>
> >>> This is Shinwoo Kim (Previously I have used cnook,
> [email protected]
> >>> contribution)
> >>>
> >>> The key compose and name have been different with xlib. So I have
> changed
> >>> the value of Shift, Control, and Alt.
> >>> Moreover, the attached patch is able to distinguish between Shift_L and
> >>> Shift_R as xlib.
> >>> Please review this and give any feedbacks, Thanks!
> >>
> >> the patch was not entirely correct. I think that I have fixed it in svn.
> >> Could you verify, please ?
> >>
> >> Vincent
> >>
> >>
> >>
> ------------------------------------------------------------------------------
> >> All the data continuously generated in your IT infrastructure
> >> contains a definitive record of customers, application performance,
> >> security threats, fraudulent activity, and more. Splunk takes this
> >> data and makes sense of it. IT sense. And common sense.
> >> http://p.sf.net/sfu/splunk-novd2d
> >> _______________________________________________
> >> enlightenment-devel mailing list
> >> [email protected]
> >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>
> >
> ------------------------------------------------------------------------------
> > All the data continuously generated in your IT infrastructure
> > contains a definitive record of customers, application performance,
> > security threats, fraudulent activity, and more. Splunk takes this
> > data and makes sense of it. IT sense. And common sense.
> > http://p.sf.net/sfu/splunk-novd2d
> > _______________________________________________
> > enlightenment-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >
> >
>
>
> ------------------------------------------------------------------------------
> All the data continuously generated in your IT infrastructure
> contains a definitive record of customers, application performance,
> security threats, fraudulent activity, and more. Splunk takes this
> data and makes sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-novd2d
> _______________________________________________
> enlightenment-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
Index: src/lib/ecore_win32/ecore_win32_event.c
===================================================================
--- src/lib/ecore_win32/ecore_win32_event.c	(revision 65466)
+++ src/lib/ecore_win32/ecore_win32_event.c	(working copy)
@@ -22,10 +22,6 @@
 {
    ECORE_WIN32_KEY_MASK_LSHIFT = 1 << 0,
    ECORE_WIN32_KEY_MASK_RSHIFT = 1 << 1,
-   ECORE_WIN32_KEY_MASK_LCONTROL = 1 << 2,
-   ECORE_WIN32_KEY_MASK_RCONTROL = 1 << 3,
-   ECORE_WIN32_KEY_MASK_LMENU = 1 << 4,
-   ECORE_WIN32_KEY_MASK_RMENU = 1 << 5
 } Ecore_Win32_Key_Mask;
 
 /***** Private declarations *****/
@@ -858,101 +854,15 @@
           break;
        }
      case VK_CONTROL:
-       {
-          SHORT res;
-
-          if (is_down)
-            {
-               res = GetKeyState(VK_LCONTROL);
-               if (res & 0x8000)
-                 {
-                    _ecore_win32_key_mask |= ECORE_WIN32_KEY_MASK_LCONTROL;
-                    kn = "Control_L";
-                    ks = "Control_L";
-                    kc = "";
-                    break;
-                 }
-               res = GetKeyState(VK_RCONTROL);
-               if (res & 0x8000)
-                 {
-                    _ecore_win32_key_mask |= ECORE_WIN32_KEY_MASK_RCONTROL;
-                    kn = "Control_R";
-                    ks = "Control_R";
-                    kc = "";
-                    break;
-                 }
-            }
-          else /* is_up */
-            {
-               res = GetKeyState(VK_LCONTROL);
-               if (!(res & 0x8000) &&
-                   (_ecore_win32_key_mask & ECORE_WIN32_KEY_MASK_LCONTROL))
-                 {
-                    kn = "Control_L";
-                    ks = "Control_L";
-                    kc = "";
-                    _ecore_win32_key_mask &= ~ECORE_WIN32_KEY_MASK_LCONTROL;
-                    break;
-                 }
-               res = GetKeyState(VK_RCONTROL);
-               if (!(res & 0x8000) &&
-                   (_ecore_win32_key_mask & ECORE_WIN32_KEY_MASK_RCONTROL))
-                 {
-                    kn = "Control_R";
-                    ks = "Control_R";
-                    kc = "";
-                    _ecore_win32_key_mask &= ~ECORE_WIN32_KEY_MASK_RCONTROL;
-                    break;
-                 }
-            }
-          break;
-       }
+       kn = "Control_L";
+       ks = "Control_L";
+       kc = "";
+       break;
      case VK_MENU:
-       {
-          SHORT res;
-
-          if (is_down)
-            {
-               res = GetKeyState(VK_LMENU);
-               if (res & 0x8000)
-                 {
-                    _ecore_win32_key_mask |= ECORE_WIN32_KEY_MASK_LMENU;
-                    kn = "Alt_L";
-                    ks = "Alt_L";
-                    kc = "";
-                 }
-               res = GetKeyState(VK_RMENU);
-               if (res & 0x8000)
-                 {
-                    _ecore_win32_key_mask |= ECORE_WIN32_KEY_MASK_RMENU;
-                    kn = "Alt_R";
-                    ks = "Alt_R";
-                    kc = "";
-                 }
-            }
-          else /* is_up */
-            {
-               res = GetKeyState(VK_LMENU);
-               if (!(res & 0x8000) &&
-                   (_ecore_win32_key_mask & ECORE_WIN32_KEY_MASK_LMENU))
-                 {
-                    kn = "Alt_L";
-                    ks = "Alt_L";
-                    kc = "";
-                    _ecore_win32_key_mask &= ~ECORE_WIN32_KEY_MASK_LMENU;
-                 }
-               res = GetKeyState(VK_RMENU);
-               if (!(res & 0x8000) &&
-                   (_ecore_win32_key_mask & ECORE_WIN32_KEY_MASK_RMENU))
-                 {
-                    kn = "Alt_R";
-                    ks = "Alt_R";
-                    kc = "";
-                    _ecore_win32_key_mask &= ~ECORE_WIN32_KEY_MASK_RMENU;
-                 }
-            }
-          break;
-       }
+       kn = "Alt_L";
+       ks = "Alt_L";
+       kc = "";
+       break;
      case VK_F1:
        kn = "F1";
        ks = "F1";
Index: src/lib/ecore_wince/ecore_wince_event.c
===================================================================
--- src/lib/ecore_wince/ecore_wince_event.c	(revision 65466)
+++ src/lib/ecore_wince/ecore_wince_event.c	(working copy)
@@ -27,10 +27,6 @@
 {
    ECORE_WINCE_KEY_MASK_LSHIFT = 1 << 0,
    ECORE_WINCE_KEY_MASK_RSHIFT = 1 << 1,
-   ECORE_WINCE_KEY_MASK_LCONTROL = 1 << 2,
-   ECORE_WINCE_KEY_MASK_RCONTROL = 1 << 3,
-   ECORE_WINCE_KEY_MASK_LMENU = 1 << 4,
-   ECORE_WINCE_KEY_MASK_RMENU = 1 << 5
 } Ecore_Wince_Key_Mask;
 
 static Ecore_WinCE_Window  *_ecore_wince_mouse_down_last_window = NULL;
@@ -310,101 +306,15 @@
           break;
        }
      case VK_CONTROL:
-       {
-          SHORT res;
-
-          if (is_down)
-            {
-               res = GetKeyState(VK_LCONTROL);
-               if (res & 0x8000)
-                 {
-                    _ecore_wince_key_mask |= ECORE_WINCE_KEY_MASK_LCONTROL;
-                    kn = "Control_L";
-                    ks = "Control_L";
-                    kc = "";
-                    break;
-                 }
-               res = GetKeyState(VK_RCONTROL);
-               if (res & 0x8000)
-                 {
-                    _ecore_wince_key_mask |= ECORE_WINCE_KEY_MASK_RCONTROL;
-                    kn = "Control_R";
-                    ks = "Control_R";
-                    kc = "";
-                    break;
-                 }
-            }
-          else /* is_up */
-            {
-               res = GetKeyState(VK_LCONTROL);
-               if (!(res & 0x8000) &&
-                   (_ecore_wince_key_mask & ECORE_WINCE_KEY_MASK_LCONTROL))
-                 {
-                    kn = "Control_L";
-                    ks = "Control_L";
-                    kc = "";
-                    _ecore_wince_key_mask &= ~ECORE_WINCE_KEY_MASK_LCONTROL;
-                    break;
-                 }
-               res = GetKeyState(VK_RCONTROL);
-               if (!(res & 0x8000) &&
-                   (_ecore_wince_key_mask & ECORE_WINCE_KEY_MASK_RCONTROL))
-                 {
-                    kn = "Control_R";
-                    ks = "Control_R";
-                    kc = "";
-                    _ecore_wince_key_mask &= ~ECORE_WINCE_KEY_MASK_RCONTROL;
-                    break;
-                 }
-            }
-          break;
-       }
+       kn = "Control_L";
+       ks = "Control_L";
+       kc = "";
+       break;
      case VK_MENU:
-       {
-          SHORT res;
-
-          if (is_down)
-            {
-               res = GetKeyState(VK_LMENU);
-               if (res & 0x8000)
-                 {
-                    _ecore_wince_key_mask |= ECORE_WINCE_KEY_MASK_LMENU;
-                    kn = "Alt_L";
-                    ks = "Alt_L";
-                    kc = "";
-                 }
-               res = GetKeyState(VK_RMENU);
-               if (res & 0x8000)
-                 {
-                    _ecore_wince_key_mask |= ECORE_WINCE_KEY_MASK_RMENU;
-                    kn = "Alt_R";
-                    ks = "Alt_R";
-                    kc = "";
-                 }
-            }
-          else /* is_up */
-            {
-               res = GetKeyState(VK_LMENU);
-               if (!(res & 0x8000) &&
-                   (_ecore_wince_key_mask & ECORE_WINCE_KEY_MASK_LMENU))
-                 {
-                    kn = "Alt_L";
-                    ks = "Alt_L";
-                    kc = "";
-                    _ecore_wince_key_mask &= ~ECORE_WINCE_KEY_MASK_LMENU;
-                 }
-               res = GetKeyState(VK_RMENU);
-               if (!(res & 0x8000) &&
-                   (_ecore_wince_key_mask & ECORE_WINCE_KEY_MASK_RMENU))
-                 {
-                    kn = "Alt_R";
-                    ks = "Alt_R";
-                    kc = "";
-                    _ecore_wince_key_mask &= ~ECORE_WINCE_KEY_MASK_RMENU;
-                 }
-            }
-          break;
-       }
+       kn = "Alt_L";
+       ks = "Alt_L";
+       kc = "";
+       break;
      default:
        /* other non keystroke characters */
        return 0;
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to