[PATCH libinput 1/7] pad: group the button state into a private struct
Yong Bakos
junk at humanoriented.com
Mon Jun 6 15:15:17 UTC 2016
Hi Peter,
> On Jun 6, 2016, at 1:50 AM, Peter Hutterer <peter.hutterer at who-t.net> wrote:
>
> This is only set on button events so use the same approach as for rings and
> strips. No functional changes.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> src/libinput.c | 14 ++++++++------
> 1 file changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/src/libinput.c b/src/libinput.c
> index 89fa594..0f676a3 100644
> --- a/src/libinput.c
> +++ b/src/libinput.c
> @@ -140,10 +140,12 @@ struct libinput_event_tablet_tool {
>
> struct libinput_event_tablet_pad {
> struct libinput_event base;
> - uint32_t button;
> - enum libinput_button_state state;
> uint64_t time;
> struct {
> + uint32_t button;
Curious, why did you choose 'button' here instead of 'id' or 'number'?
The semantics would be a little cleaner, especially below, when
returning `button.button` rather than, say, `button.number`.
> + enum libinput_button_state state;
> + } button;
> + struct {
> enum libinput_tablet_pad_ring_axis_source source;
> double position;
> int number;
> @@ -2408,8 +2410,8 @@ tablet_pad_notify_button(struct libinput_device *device,
>
> *button_event = (struct libinput_event_tablet_pad) {
> .time = time,
> - .button = button,
> - .state = state,
> + .button.button = button,
> + .button.state = state,
> };
>
> post_device_event(device,
> @@ -2973,7 +2975,7 @@ libinput_event_tablet_pad_get_button_number(struct libinput_event_tablet_pad *ev
> 0,
> LIBINPUT_EVENT_TABLET_PAD_BUTTON);
>
> - return event->button;
> + return event->button.button;
^ Per my comment above.
yong
> }
>
> LIBINPUT_EXPORT enum libinput_button_state
> @@ -2984,7 +2986,7 @@ libinput_event_tablet_pad_get_button_state(struct libinput_event_tablet_pad *eve
> LIBINPUT_BUTTON_STATE_RELEASED,
> LIBINPUT_EVENT_TABLET_PAD_BUTTON);
>
> - return event->state;
> + return event->button.state;
> }
>
> LIBINPUT_EXPORT uint32_t
> --
> 2.7.4
More information about the wayland-devel
mailing list