From e6ad67911bffcd804549bd1812a77a305cc14eb4 Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 14:54:53 +0200 Subject: Refactor out wlr_input_device_state --- include/wlr/interfaces/wlr_input_device.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/include/wlr/interfaces/wlr_input_device.h b/include/wlr/interfaces/wlr_input_device.h index 26729c7c..b236a24a 100644 --- a/include/wlr/interfaces/wlr_input_device.h +++ b/include/wlr/interfaces/wlr_input_device.h @@ -3,13 +3,13 @@ #include struct wlr_input_device_impl { - void (*destroy)(struct wlr_input_device_state *state); + void (*destroy)(struct wlr_input_device *wlr_device); }; -struct wlr_input_device *wlr_input_device_create( +void wlr_input_device_init( + struct wlr_input_device *wlr_device, enum wlr_input_device_type type, struct wlr_input_device_impl *impl, - struct wlr_input_device_state *state, const char *name, int vendor, int product); void wlr_input_device_destroy(struct wlr_input_device *dev); -- cgit v1.2.3 From 0a3246ad979d768933a68b81e738853bd7c964a2 Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 15:41:14 +0200 Subject: Refactor out wlr_keyboard_state --- backend/libinput/keyboard.c | 25 ++++++++++++++++--------- backend/wayland/wl_seat.c | 8 +++++++- include/wlr/interfaces/wlr_keyboard.h | 7 +++---- include/wlr/types/wlr_keyboard.h | 2 -- types/wlr_keyboard.c | 18 ++++++++---------- 5 files changed, 34 insertions(+), 26 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/backend/libinput/keyboard.c b/backend/libinput/keyboard.c index b231828d..db4ac0ed 100644 --- a/backend/libinput/keyboard.c +++ b/backend/libinput/keyboard.c @@ -7,17 +7,21 @@ #include #include "backend/libinput.h" -struct wlr_keyboard_state { +struct wlr_libinput_keyboard { + struct wlr_keyboard wlr_keyboard; struct libinput_device *libinput_dev; }; -static void wlr_libinput_keyboard_set_leds(struct wlr_keyboard_state *kbstate, uint32_t leds) { - libinput_device_led_update(kbstate->libinput_dev, leds); +static void wlr_libinput_keyboard_set_leds(struct wlr_keyboard *wlr_kb, uint32_t leds) { + struct wlr_libinput_keyboard *wlr_libinput_kb = (struct wlr_libinput_keyboard *)wlr_kb; + libinput_device_led_update(wlr_libinput_kb->libinput_dev, leds); } -static void wlr_libinput_keyboard_destroy(struct wlr_keyboard_state *kbstate) { - libinput_device_unref(kbstate->libinput_dev); - free(kbstate); +static void wlr_libinput_keyboard_destroy(struct wlr_keyboard *wlr_kb) { + struct wlr_libinput_keyboard *wlr_libinput_kb = + (struct wlr_libinput_keyboard *)wlr_kb; + libinput_device_unref(wlr_libinput_kb->libinput_dev); + free(wlr_libinput_kb); } struct wlr_keyboard_impl impl = { @@ -28,11 +32,14 @@ struct wlr_keyboard_impl impl = { struct wlr_keyboard *wlr_libinput_keyboard_create( struct libinput_device *libinput_dev) { assert(libinput_dev); - struct wlr_keyboard_state *kbstate = calloc(1, sizeof(struct wlr_keyboard_state)); - kbstate->libinput_dev = libinput_dev; + struct wlr_libinput_keyboard *wlr_libinput_kb = + calloc(1, sizeof(struct wlr_libinput_keyboard)); + wlr_libinput_kb->libinput_dev = libinput_dev; libinput_device_ref(libinput_dev); libinput_device_led_update(libinput_dev, 0); - return wlr_keyboard_create(&impl, kbstate); + struct wlr_keyboard *wlr_kb = &wlr_libinput_kb->wlr_keyboard; + wlr_keyboard_init(wlr_kb, &impl); + return wlr_kb; } void handle_keyboard_key(struct libinput_event *event, diff --git a/backend/wayland/wl_seat.c b/backend/wayland/wl_seat.c index 9b54a328..7afe8775 100644 --- a/backend/wayland/wl_seat.c +++ b/backend/wayland/wl_seat.c @@ -237,12 +237,18 @@ static void seat_handle_capabilities(void *data, struct wl_seat *wl_seat, wlr_log(L_ERROR, "Unable to allocate wl_pointer device"); return; } + wlr_device->keyboard = calloc(1, sizeof(struct wlr_keyboard)); + if (!wlr_device->keyboard) { + free(wlr_device); + wlr_log(L_ERROR, "Unable to allocate wlr keyboard"); + return; + } + wlr_keyboard_init(wlr_device->keyboard, NULL); struct wlr_wl_input_device *wlr_wl_device = (struct wlr_wl_input_device *)wlr_device; struct wl_keyboard *wl_keyboard = wl_seat_get_keyboard(wl_seat); wl_keyboard_add_listener(wl_keyboard, &keyboard_listener, wlr_device); - wlr_device->keyboard = wlr_keyboard_create(NULL, NULL); wlr_wl_device->resource = wl_keyboard; wl_signal_emit(&backend->backend.events.input_add, wlr_device); } diff --git a/include/wlr/interfaces/wlr_keyboard.h b/include/wlr/interfaces/wlr_keyboard.h index 1acc6428..779b302f 100644 --- a/include/wlr/interfaces/wlr_keyboard.h +++ b/include/wlr/interfaces/wlr_keyboard.h @@ -4,12 +4,11 @@ #include struct wlr_keyboard_impl { - void (*destroy)(struct wlr_keyboard_state *state); - void (*led_update)(struct wlr_keyboard_state *state, uint32_t leds); + void (*destroy)(struct wlr_keyboard *keyboard); + void (*led_update)(struct wlr_keyboard *keyboard, uint32_t leds); }; -struct wlr_keyboard *wlr_keyboard_create(struct wlr_keyboard_impl *impl, - struct wlr_keyboard_state *state); +void wlr_keyboard_init(struct wlr_keyboard *keyboard, struct wlr_keyboard_impl *impl); void wlr_keyboard_destroy(struct wlr_keyboard *keyboard); #endif diff --git a/include/wlr/types/wlr_keyboard.h b/include/wlr/types/wlr_keyboard.h index bef064e5..ce7d6659 100644 --- a/include/wlr/types/wlr_keyboard.h +++ b/include/wlr/types/wlr_keyboard.h @@ -10,11 +10,9 @@ enum WLR_KEYBOARD_LED { WLR_LED_LAST }; -struct wlr_keyboard_state; struct wlr_keyboard_impl; struct wlr_keyboard { - struct wlr_keyboard_state *state; struct wlr_keyboard_impl *impl; struct { diff --git a/types/wlr_keyboard.c b/types/wlr_keyboard.c index 26dd2977..14930f6f 100644 --- a/types/wlr_keyboard.c +++ b/types/wlr_keyboard.c @@ -4,25 +4,23 @@ #include #include -struct wlr_keyboard *wlr_keyboard_create(struct wlr_keyboard_impl *impl, - struct wlr_keyboard_state *state) { - struct wlr_keyboard *kb = calloc(1, sizeof(struct wlr_keyboard)); +void wlr_keyboard_init(struct wlr_keyboard *kb, + struct wlr_keyboard_impl *impl) { kb->impl = impl; - kb->state = state; wl_signal_init(&kb->events.key); - return kb; } void wlr_keyboard_destroy(struct wlr_keyboard *kb) { if (!kb) return; - if (kb->impl) { - kb->impl->destroy(kb->state); + if (kb->impl && kb->impl->destroy) { + kb->impl->destroy(kb); + } else { + free(kb); } - free(kb); } void wlr_keyboard_led_update(struct wlr_keyboard *kb, uint32_t leds) { - if (kb->impl) { - kb->impl->led_update(kb->state, leds); + if (kb->impl && kb->impl->led_update) { + kb->impl->led_update(kb, leds); } } -- cgit v1.2.3 From 53e7bebd2381318b9edcbc8d305d4ec2bed9b3ce Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 15:55:48 +0200 Subject: Refactor out wlr_pointer_state --- backend/libinput/pointer.c | 8 ++++++- backend/wayland/wl_seat.c | 46 +++++++++++++++++------------------- include/backend/wayland.h | 3 ++- include/wlr/interfaces/wlr_pointer.h | 6 ++--- include/wlr/types/wlr_pointer.h | 2 -- types/wlr_pointer.c | 14 +++++------ 6 files changed, 40 insertions(+), 39 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/backend/libinput/pointer.c b/backend/libinput/pointer.c index c2c47592..8bda205d 100644 --- a/backend/libinput/pointer.c +++ b/backend/libinput/pointer.c @@ -10,7 +10,13 @@ struct wlr_pointer *wlr_libinput_pointer_create( struct libinput_device *libinput_dev) { assert(libinput_dev); - return wlr_pointer_create(NULL, NULL); + struct wlr_pointer *wlr_pointer = calloc(1, sizeof(struct wlr_pointer)); + if (!wlr_pointer) { + wlr_log(L_ERROR, "Unable to allocate wlr_pointer"); + return NULL; + } + wlr_pointer_init(wlr_pointer, NULL); + return wlr_pointer; } void handle_pointer_motion(struct libinput_event *event, diff --git a/backend/wayland/wl_seat.c b/backend/wayland/wl_seat.c index 7afe8775..010566bb 100644 --- a/backend/wayland/wl_seat.c +++ b/backend/wayland/wl_seat.c @@ -17,31 +17,33 @@ static void pointer_handle_enter(void *data, struct wl_pointer *wl_pointer, wl_fixed_t surface_y) { struct wlr_input_device *dev = data; struct wlr_wl_input_device *wlr_wl_dev = (struct wlr_wl_input_device *)dev; - assert(dev && dev->pointer && dev->pointer->state); + assert(dev && dev->pointer); + struct wlr_wl_pointer *wlr_wl_pointer = (struct wlr_wl_pointer *)dev->pointer; struct wlr_wl_backend_output* output = wlr_wl_output_for_surface(wlr_wl_dev->backend, surface); assert(output); - dev->pointer->state->current_output = output; + wlr_wl_pointer->current_output = output; } static void pointer_handle_leave(void *data, struct wl_pointer *wl_pointer, uint32_t serial, struct wl_surface *surface) { struct wlr_input_device *dev = data; - assert(dev && dev->pointer && dev->pointer->state); - dev->pointer->state->current_output = NULL; + assert(dev && dev->pointer); + struct wlr_wl_pointer *wlr_wl_pointer = (struct wlr_wl_pointer *)dev->pointer; + wlr_wl_pointer->current_output = NULL; } static void pointer_handle_motion(void *data, struct wl_pointer *wl_pointer, uint32_t time, wl_fixed_t surface_x, wl_fixed_t surface_y) { struct wlr_input_device *dev = data; - assert(dev && dev->pointer && dev->pointer->state); - struct wlr_pointer_state *state = dev->pointer->state; - if (!state->current_output) { + assert(dev && dev->pointer); + struct wlr_wl_pointer *wlr_wl_pointer = (struct wlr_wl_pointer *)dev->pointer; + if (!wlr_wl_pointer->current_output) { wlr_log(L_ERROR, "pointer motion event without current output"); return; } int width, height; - wl_egl_window_get_attached_size(state->current_output->egl_window, + wl_egl_window_get_attached_size(wlr_wl_pointer->current_output->egl_window, &width, &height); struct wlr_event_pointer_motion_absolute wlr_event; wlr_event.time_sec = time / 1000; @@ -70,13 +72,14 @@ static void pointer_handle_axis(void *data, struct wl_pointer *wl_pointer, uint32_t time, uint32_t axis, wl_fixed_t value) { struct wlr_input_device *dev = data; assert(dev && dev->pointer); + struct wlr_wl_pointer *wlr_wl_pointer = (struct wlr_wl_pointer *)dev->pointer; struct wlr_event_pointer_axis wlr_event; wlr_event.delta = value; wlr_event.orientation = axis; wlr_event.time_sec = time / 1000; wlr_event.time_usec = time * 1000; - wlr_event.source = dev->pointer->state->axis_source; + wlr_event.source = wlr_wl_pointer->axis_source; wl_signal_emit(&dev->pointer->events.axis, &wlr_event); } @@ -87,8 +90,10 @@ static void pointer_handle_frame(void *data, struct wl_pointer *wl_pointer) { static void pointer_handle_axis_source(void *data, struct wl_pointer *wl_pointer, uint32_t axis_source) { struct wlr_input_device *dev = data; - assert(dev && dev->pointer && dev->pointer->state); - dev->pointer->state->axis_source = axis_source; + assert(dev && dev->pointer); + struct wlr_wl_pointer *wlr_wl_pointer = (struct wlr_wl_pointer *)dev->pointer; + + wlr_wl_pointer->axis_source = axis_source; } static void pointer_handle_axis_stop(void *data, struct wl_pointer *wl_pointer, @@ -173,14 +178,6 @@ static struct wlr_input_device_impl input_device_impl = { .destroy = input_device_destroy }; -static void pointer_destroy(struct wlr_pointer_state *state) { - free(state); -} - -static struct wlr_pointer_impl pointer_impl = { - .destroy = pointer_destroy -}; - static struct wlr_input_device *allocate_device(struct wlr_wl_backend *backend, enum wlr_input_device_type type) { struct wlr_wl_input_device *wlr_wl_dev; @@ -208,15 +205,15 @@ static void seat_handle_capabilities(void *data, struct wl_seat *wl_seat, if ((caps & WL_SEAT_CAPABILITY_POINTER)) { wlr_log(L_DEBUG, "seat %p offered pointer", (void*) wl_seat); - struct wlr_pointer_state *pointer_state; - if (!(pointer_state = calloc(1, sizeof(struct wlr_pointer_state)))) { - wlr_log(L_ERROR, "Unable to allocate wlr_pointer_state"); + struct wlr_wl_pointer *wlr_wl_pointer; + if (!(wlr_wl_pointer = calloc(1, sizeof(struct wlr_wl_pointer)))) { + wlr_log(L_ERROR, "Unable to allocate wlr_wl_pointer"); return; } struct wlr_input_device *wlr_device; if (!(wlr_device = allocate_device(backend, WLR_INPUT_DEVICE_POINTER))) { - free(pointer_state); + free(wlr_wl_pointer); wlr_log(L_ERROR, "Unable to allocate wlr_device for pointer"); return; } @@ -225,7 +222,8 @@ static void seat_handle_capabilities(void *data, struct wl_seat *wl_seat, struct wl_pointer *wl_pointer = wl_seat_get_pointer(wl_seat); wl_pointer_add_listener(wl_pointer, &pointer_listener, wlr_device); - wlr_device->pointer = wlr_pointer_create(&pointer_impl, pointer_state); + wlr_device->pointer = &wlr_wl_pointer->wlr_pointer; + wlr_pointer_init(wlr_device->pointer, NULL); wlr_wl_device->resource = wl_pointer; wl_signal_emit(&backend->backend.events.input_add, wlr_device); } diff --git a/include/backend/wayland.h b/include/backend/wayland.h index 6d4b0138..d930f21b 100644 --- a/include/backend/wayland.h +++ b/include/backend/wayland.h @@ -48,7 +48,8 @@ struct wlr_wl_input_device { void *resource; }; -struct wlr_pointer_state { +struct wlr_wl_pointer { + struct wlr_pointer wlr_pointer; enum wlr_axis_source axis_source; struct wlr_wl_backend_output *current_output; }; diff --git a/include/wlr/interfaces/wlr_pointer.h b/include/wlr/interfaces/wlr_pointer.h index 8c3f7e0d..8d4bf703 100644 --- a/include/wlr/interfaces/wlr_pointer.h +++ b/include/wlr/interfaces/wlr_pointer.h @@ -3,11 +3,11 @@ #include struct wlr_pointer_impl { - void (*destroy)(struct wlr_pointer_state *state); + void (*destroy)(struct wlr_pointer *pointer); }; -struct wlr_pointer *wlr_pointer_create(struct wlr_pointer_impl *impl, - struct wlr_pointer_state *state); +void wlr_pointer_init(struct wlr_pointer *pointer, + struct wlr_pointer_impl *impl); void wlr_pointer_destroy(struct wlr_pointer *pointer); #endif diff --git a/include/wlr/types/wlr_pointer.h b/include/wlr/types/wlr_pointer.h index 139ba868..13a2d045 100644 --- a/include/wlr/types/wlr_pointer.h +++ b/include/wlr/types/wlr_pointer.h @@ -4,11 +4,9 @@ #include #include -struct wlr_pointer_state; struct wlr_pointer_impl; struct wlr_pointer { - struct wlr_pointer_state *state; struct wlr_pointer_impl *impl; struct { diff --git a/types/wlr_pointer.c b/types/wlr_pointer.c index fc1ae808..02c2ce61 100644 --- a/types/wlr_pointer.c +++ b/types/wlr_pointer.c @@ -4,22 +4,20 @@ #include #include -struct wlr_pointer *wlr_pointer_create(struct wlr_pointer_impl *impl, - struct wlr_pointer_state *state) { - struct wlr_pointer *pointer = calloc(1, sizeof(struct wlr_pointer)); +void wlr_pointer_init(struct wlr_pointer *pointer, + struct wlr_pointer_impl *impl) { pointer->impl = impl; - pointer->state = state; wl_signal_init(&pointer->events.motion); wl_signal_init(&pointer->events.motion_absolute); wl_signal_init(&pointer->events.button); wl_signal_init(&pointer->events.axis); - return pointer; } void wlr_pointer_destroy(struct wlr_pointer *pointer) { if (!pointer) return; - if (pointer->impl) { - pointer->impl->destroy(pointer->state); + if (pointer->impl && pointer->impl->destroy) { + pointer->impl->destroy(pointer); + } else { + free(pointer); } - free(pointer); } -- cgit v1.2.3 From a289940bff8bcda5cc6aeb1d88f4f1e5c2c38317 Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 15:59:59 +0200 Subject: Refactor out wlr_tablet_pad_state --- backend/libinput/tablet_pad.c | 8 +++++++- include/wlr/interfaces/wlr_tablet_pad.h | 6 +++--- include/wlr/types/wlr_tablet_pad.h | 2 -- types/wlr_tablet_pad.c | 14 ++++++-------- 4 files changed, 16 insertions(+), 14 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/backend/libinput/tablet_pad.c b/backend/libinput/tablet_pad.c index 9713bb1d..efad1068 100644 --- a/backend/libinput/tablet_pad.c +++ b/backend/libinput/tablet_pad.c @@ -10,7 +10,13 @@ struct wlr_tablet_pad *wlr_libinput_tablet_pad_create( struct libinput_device *libinput_dev) { assert(libinput_dev); - return wlr_tablet_pad_create(NULL, NULL); + struct wlr_tablet_pad *wlr_tablet_pad = calloc(1, sizeof(struct wlr_tablet_pad)); + if (!wlr_tablet_pad) { + wlr_log(L_ERROR, "Unable to allocate wlr_tablet_pad"); + return NULL; + } + wlr_tablet_pad_init(wlr_tablet_pad, NULL); + return wlr_tablet_pad; } void handle_tablet_pad_button(struct libinput_event *event, diff --git a/include/wlr/interfaces/wlr_tablet_pad.h b/include/wlr/interfaces/wlr_tablet_pad.h index 09274c6c..81af3c3f 100644 --- a/include/wlr/interfaces/wlr_tablet_pad.h +++ b/include/wlr/interfaces/wlr_tablet_pad.h @@ -3,11 +3,11 @@ #include struct wlr_tablet_pad_impl { - void (*destroy)(struct wlr_tablet_pad_state *pad); + void (*destroy)(struct wlr_tablet_pad *pad); }; -struct wlr_tablet_pad *wlr_tablet_pad_create(struct wlr_tablet_pad_impl *impl, - struct wlr_tablet_pad_state *state); +void wlr_tablet_pad_init(struct wlr_tablet_pad *pad, + struct wlr_tablet_pad_impl *impl); void wlr_tablet_pad_destroy(struct wlr_tablet_pad *pad); #endif diff --git a/include/wlr/types/wlr_tablet_pad.h b/include/wlr/types/wlr_tablet_pad.h index d2365086..6c14c669 100644 --- a/include/wlr/types/wlr_tablet_pad.h +++ b/include/wlr/types/wlr_tablet_pad.h @@ -11,11 +11,9 @@ */ struct wlr_tablet_pad_impl; -struct wlr_tablet_pad_state; struct wlr_tablet_pad { struct wlr_tablet_pad_impl *impl; - struct wlr_tablet_pad_state *state; struct { struct wl_signal button; diff --git a/types/wlr_tablet_pad.c b/types/wlr_tablet_pad.c index 80c6187d..b7f03493 100644 --- a/types/wlr_tablet_pad.c +++ b/types/wlr_tablet_pad.c @@ -4,21 +4,19 @@ #include #include -struct wlr_tablet_pad *wlr_tablet_pad_create(struct wlr_tablet_pad_impl *impl, - struct wlr_tablet_pad_state *state) { - struct wlr_tablet_pad *pad = calloc(1, sizeof(struct wlr_tablet_pad)); +void wlr_tablet_pad_init(struct wlr_tablet_pad *pad, + struct wlr_tablet_pad_impl *impl) { pad->impl = impl; - pad->state = state; wl_signal_init(&pad->events.button); wl_signal_init(&pad->events.ring); wl_signal_init(&pad->events.strip); - return pad; } void wlr_tablet_pad_destroy(struct wlr_tablet_pad *pad) { if (!pad) return; - if (pad->impl) { - pad->impl->destroy(pad->state); + if (pad->impl && pad->impl->destroy) { + pad->impl->destroy(pad); + } else { + free(pad); } - free(pad); } -- cgit v1.2.3 From bd67849c03a13b2a12d35fb068d30eb0f8ea3cc9 Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 16:03:05 +0200 Subject: Refactor out wlr_tablet_tool_state --- backend/libinput/tablet_tool.c | 8 +++++++- include/wlr/interfaces/wlr_tablet_tool.h | 6 +++--- include/wlr/types/wlr_tablet_tool.h | 2 -- types/wlr_tablet_tool.c | 14 ++++++-------- 4 files changed, 16 insertions(+), 14 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/backend/libinput/tablet_tool.c b/backend/libinput/tablet_tool.c index efe77f0a..8b3d34ed 100644 --- a/backend/libinput/tablet_tool.c +++ b/backend/libinput/tablet_tool.c @@ -10,7 +10,13 @@ struct wlr_tablet_tool *wlr_libinput_tablet_tool_create( struct libinput_device *libinput_dev) { assert(libinput_dev); - return wlr_tablet_tool_create(NULL, NULL); + struct wlr_tablet_tool *wlr_tablet_tool = calloc(1, sizeof(struct wlr_tablet_tool)); + if (!wlr_tablet_tool) { + wlr_log(L_ERROR, "Unable to allocate wlr_tablet_tool"); + return NULL; + } + wlr_tablet_tool_init(wlr_tablet_tool, NULL); + return wlr_tablet_tool; } void handle_tablet_tool_axis(struct libinput_event *event, diff --git a/include/wlr/interfaces/wlr_tablet_tool.h b/include/wlr/interfaces/wlr_tablet_tool.h index cd326878..43a24fd0 100644 --- a/include/wlr/interfaces/wlr_tablet_tool.h +++ b/include/wlr/interfaces/wlr_tablet_tool.h @@ -3,11 +3,11 @@ #include struct wlr_tablet_tool_impl { - void (*destroy)(struct wlr_tablet_tool_state *tool); + void (*destroy)(struct wlr_tablet_tool *tool); }; -struct wlr_tablet_tool *wlr_tablet_tool_create(struct wlr_tablet_tool_impl *impl, - struct wlr_tablet_tool_state *state); +void wlr_tablet_tool_init(struct wlr_tablet_tool *tool, + struct wlr_tablet_tool_impl *impl); void wlr_tablet_tool_destroy(struct wlr_tablet_tool *tool); #endif diff --git a/include/wlr/types/wlr_tablet_tool.h b/include/wlr/types/wlr_tablet_tool.h index f99cd065..dcb9c191 100644 --- a/include/wlr/types/wlr_tablet_tool.h +++ b/include/wlr/types/wlr_tablet_tool.h @@ -5,11 +5,9 @@ #include struct wlr_tablet_tool_impl; -struct wlr_tablet_tool_state; struct wlr_tablet_tool { struct wlr_tablet_tool_impl *impl; - struct wlr_tablet_tool_state *state; struct { struct wl_signal axis; diff --git a/types/wlr_tablet_tool.c b/types/wlr_tablet_tool.c index 1e939c3b..9f12a5ba 100644 --- a/types/wlr_tablet_tool.c +++ b/types/wlr_tablet_tool.c @@ -4,22 +4,20 @@ #include #include -struct wlr_tablet_tool *wlr_tablet_tool_create(struct wlr_tablet_tool_impl *impl, - struct wlr_tablet_tool_state *state) { - struct wlr_tablet_tool *tool = calloc(1, sizeof(struct wlr_tablet_tool)); +void wlr_tablet_tool_init(struct wlr_tablet_tool *tool, + struct wlr_tablet_tool_impl *impl) { tool->impl = impl; - tool->state = state; wl_signal_init(&tool->events.axis); wl_signal_init(&tool->events.proximity); wl_signal_init(&tool->events.tip); wl_signal_init(&tool->events.button); - return tool; } void wlr_tablet_tool_destroy(struct wlr_tablet_tool *tool) { if (!tool) return; - if (tool->impl) { - tool->impl->destroy(tool->state); + if (tool->impl && tool->impl->destroy) { + tool->impl->destroy(tool); + } else { + free(tool); } - free(tool); } -- cgit v1.2.3 From ba20d5b3cae29a90b1d66c9d53f295a7c41c932e Mon Sep 17 00:00:00 2001 From: Dominique Martinet Date: Mon, 14 Aug 2017 16:07:00 +0200 Subject: Refactor out wlr_touch_state --- backend/libinput/touch.c | 8 +++++++- include/wlr/interfaces/wlr_touch.h | 6 +++--- include/wlr/types/wlr_touch.h | 2 -- types/wlr_touch.c | 14 ++++++-------- 4 files changed, 16 insertions(+), 14 deletions(-) (limited to 'include/wlr/interfaces') diff --git a/backend/libinput/touch.c b/backend/libinput/touch.c index 5d990d3c..9e08d028 100644 --- a/backend/libinput/touch.c +++ b/backend/libinput/touch.c @@ -10,7 +10,13 @@ struct wlr_touch *wlr_libinput_touch_create( struct libinput_device *libinput_dev) { assert(libinput_dev); - return wlr_touch_create(NULL, NULL); + struct wlr_touch *wlr_touch = calloc(1, sizeof(struct wlr_touch)); + if (!wlr_touch) { + wlr_log(L_ERROR, "Unable to allocate wlr_touch"); + return NULL; + } + wlr_touch_init(wlr_touch, NULL); + return wlr_touch; } void handle_touch_down(struct libinput_event *event, diff --git a/include/wlr/interfaces/wlr_touch.h b/include/wlr/interfaces/wlr_touch.h index d83ad558..b5fcef18 100644 --- a/include/wlr/interfaces/wlr_touch.h +++ b/include/wlr/interfaces/wlr_touch.h @@ -3,11 +3,11 @@ #include struct wlr_touch_impl { - void (*destroy)(struct wlr_touch_state *state); + void (*destroy)(struct wlr_touch *touch); }; -struct wlr_touch *wlr_touch_create(struct wlr_touch_impl *impl, - struct wlr_touch_state *state); +void wlr_touch_init(struct wlr_touch *touch, + struct wlr_touch_impl *impl); void wlr_touch_destroy(struct wlr_touch *touch); #endif diff --git a/include/wlr/types/wlr_touch.h b/include/wlr/types/wlr_touch.h index 740d70f6..93069fcb 100644 --- a/include/wlr/types/wlr_touch.h +++ b/include/wlr/types/wlr_touch.h @@ -3,11 +3,9 @@ #include #include -struct wlr_touch_state; struct wlr_touch_impl; struct wlr_touch { - struct wlr_touch_state *state; struct wlr_touch_impl *impl; struct { diff --git a/types/wlr_touch.c b/types/wlr_touch.c index e698decd..46ae7ce9 100644 --- a/types/wlr_touch.c +++ b/types/wlr_touch.c @@ -4,22 +4,20 @@ #include #include -struct wlr_touch *wlr_touch_create(struct wlr_touch_impl *impl, - struct wlr_touch_state *state) { - struct wlr_touch *touch = calloc(1, sizeof(struct wlr_touch)); +void wlr_touch_init(struct wlr_touch *touch, + struct wlr_touch_impl *impl) { touch->impl = impl; - touch->state = state; wl_signal_init(&touch->events.down); wl_signal_init(&touch->events.up); wl_signal_init(&touch->events.motion); wl_signal_init(&touch->events.cancel); - return touch; } void wlr_touch_destroy(struct wlr_touch *touch) { if (!touch) return; - if (touch->impl) { - touch->impl->destroy(touch->state); + if (touch->impl && touch->impl->destroy) { + touch->impl->destroy(touch); + } else { + free(touch); } - free(touch); } -- cgit v1.2.3