Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-11-16 Thread Daniel Stone
Hi, On 2 September 2016 at 09:30, Giulio Camuffo wrote: > 2016-09-02 10:27 GMT+02:00 Olivier Fourdan : >>> 2016-09-01 11:52 GMT+02:00 Olivier Fourdan : >>> >> I’m ok with that, but why not updating all three "grab_*" members? >>> > >>> > While I think the key itself would be OK, I suspect updatin

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-09-02 Thread Giulio Camuffo
2016-09-02 10:27 GMT+02:00 Olivier Fourdan : > Hi, > > - Original Message - >> 2016-09-01 11:52 GMT+02:00 Olivier Fourdan : >> > Hi >> > >> >> > + keyboard->grab_serial = >> >> > wl_display_get_serial(compositor->wl_display); >> >> > if (state == WL_KEYBOARD_KEY_STATE_PRESSED) { >> >>

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-09-02 Thread Olivier Fourdan
Hi, - Original Message - > 2016-09-01 11:52 GMT+02:00 Olivier Fourdan : > > Hi > > > >> > + keyboard->grab_serial = > >> > wl_display_get_serial(compositor->wl_display); > >> > if (state == WL_KEYBOARD_KEY_STATE_PRESSED) { > >> > - keyboard->grab_serial = > >> > -

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-09-02 Thread Giulio Camuffo
2016-09-01 11:52 GMT+02:00 Olivier Fourdan : > Hi > >> > + keyboard->grab_serial = wl_display_get_serial(compositor->wl_display); >> > if (state == WL_KEYBOARD_KEY_STATE_PRESSED) { >> > - keyboard->grab_serial = >> > - wl_display_get_serial(compositor->wl_display);

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-09-01 Thread Olivier Fourdan
Hi > > + keyboard->grab_serial = wl_display_get_serial(compositor->wl_display); > > if (state == WL_KEYBOARD_KEY_STATE_PRESSED) { > > - keyboard->grab_serial = > > - wl_display_get_serial(compositor->wl_display); > > keyboard->grab_time = time; > >

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-08-31 Thread Quentin Glidic
On 30/06/2016 16:01, Olivier Fourdan wrote: Other compositors such as mutter update the keyboard serial for both key press and key release, unlike weston which updates it only on key press. When dealing with popup windows which require a match in serials, if the event that caused the popup to be

Re: [PATCH weston 1/2] input: Update keyboard serial on press and release

2016-08-10 Thread Olivier Fourdan
Hi all, Small reminder, I don't think I got any review/reply of these two patches... Cheers, Olivier - Original Message - > Other compositors such as mutter update the keyboard serial for both key > press and key release, unlike weston which updates it only on key press. > > When deali

[PATCH weston 1/2] input: Update keyboard serial on press and release

2016-06-30 Thread Olivier Fourdan
Other compositors such as mutter update the keyboard serial for both key press and key release, unlike weston which updates it only on key press. When dealing with popup windows which require a match in serials, if the event that caused the popup to be shown is a key release, then the popup would