On 3 March 2017 at 23:05, Daniel Stone <dani...@collabora.com> wrote:

> -       x = (x - output->base.x) * output->base.current_scale;
> -       y = (y - output->base.y) * output->base.current_scale;
> +       x = (output->cursor_plane.x - output->base.x) *
> +               output->base.current_scale;
> +       y = (output->cursor_plane.y - output->base.y) *
> +               output->base.current_scale;
>
> -       if (output->cursor_plane.x != x || output->cursor_plane.y != y) {
As we're dropping this, won't we end up with lots of unnecessary calls
to drmModeMoveCursor() ?
Is there something which will help us ?

> -               if (drmModeMoveCursor(b->drm.fd, output->crtc_id, x, y)) {
> -                       weston_log("failed to move cursor: %m\n");
> -                       b->cursors_are_broken = 1;
> -               }
> -
> -               output->cursor_plane.x = x;
> -               output->cursor_plane.y = y;
> +       if (drmModeMoveCursor(b->drm.fd, output->crtc_id, x, y)) {
> +               weston_log("failed to move cursor: %m\n");
> +               b->cursors_are_broken = 1;


-Emil
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to