diff options
author | Simon Ser <contact@emersion.fr> | 2022-05-27 18:32:50 +0200 |
---|---|---|
committer | Isaac Freund <mail@isaacfreund.com> | 2022-05-31 11:40:47 +0000 |
commit | 1ab3e582acda1829e2ad84aa278550c5cfa174b1 (patch) | |
tree | 2cd3c83ac7556632ce1e4fa4ba0ff0a2969e4b37 | |
parent | 459a642e83349cc96b722450c391a13758eeb1a6 (diff) |
output: re-implement set_image with set_buffer
Avoids having two code-paths doing the same thing.
-rw-r--r-- | types/output/cursor.c | 41 |
1 files changed, 9 insertions, 32 deletions
diff --git a/types/output/cursor.c b/types/output/cursor.c index 0d82c354..2cbf67f5 100644 --- a/types/output/cursor.c +++ b/types/output/cursor.c @@ -8,6 +8,7 @@ #include <wlr/util/log.h> #include "render/allocator/allocator.h" #include "render/swapchain.h" +#include "types/wlr_buffer.h" #include "types/wlr_output.h" #include "util/signal.h" @@ -375,40 +376,16 @@ static bool output_cursor_attempt_hardware(struct wlr_output_cursor *cursor) { bool wlr_output_cursor_set_image(struct wlr_output_cursor *cursor, const uint8_t *pixels, int32_t stride, uint32_t width, uint32_t height, int32_t hotspot_x, int32_t hotspot_y) { - struct wlr_renderer *renderer = cursor->output->renderer; - if (!renderer) { + struct wlr_readonly_data_buffer *buffer = + readonly_data_buffer_create(DRM_FORMAT_ARGB8888, stride, width, height, + pixels); + if (buffer == NULL) { return false; } - - output_cursor_reset(cursor); - - cursor->width = width; - cursor->height = height; - cursor->hotspot_x = hotspot_x; - cursor->hotspot_y = hotspot_y; - output_cursor_update_visible(cursor); - - wlr_texture_destroy(cursor->texture); - cursor->texture = NULL; - - cursor->enabled = false; - if (pixels != NULL) { - cursor->texture = wlr_texture_from_pixels(renderer, - DRM_FORMAT_ARGB8888, stride, width, height, pixels); - if (cursor->texture == NULL) { - return false; - } - cursor->enabled = true; - } - - if (output_cursor_attempt_hardware(cursor)) { - return true; - } - - wlr_log(WLR_DEBUG, "Falling back to software cursor on output '%s'", - cursor->output->name); - output_cursor_damage_whole(cursor); - return true; + bool ok = wlr_output_cursor_set_buffer(cursor, &buffer->base, + hotspot_x, hotspot_y); + wlr_buffer_drop(&buffer->base); + return ok; } bool wlr_output_cursor_set_buffer(struct wlr_output_cursor *cursor, |