[PATCH weston] compositor-drm: Don't drop viewported surfaces into the cursor plane

Daniel Stone daniel at fooishbar.org
Fri Sep 30 19:39:59 UTC 2016


Hi Derek,

On Friday, 30 September 2016, Derek Foreman <derekf at osg.samsung.com> wrote:
>
> +static bool
> +viewport_is_cursor_compatible(struct weston_buffer_viewport *viewport)
> +{
> +       /* While we could technically relax soem of these constraints
> +        * if we implemented cropping in cursor_bo_update, it doesn't
> +        * seem worth the effort.
> +        * It is also possible to more thoroughly test if src and dst
> +        * sizes match when widths aren't -1, but again, likely not
> +        * worth the complexity.
> +        */
> +       if (viewport->surface.width != -1)
> +               return false;
> +       if (wl_fixed_to_int(viewport->buffer.src_x) != 0)
> +               return false;
> +       if (wl_fixed_to_int(viewport->buffer.src_y) != 0)
> +               return false;
> +       if (wl_fixed_to_int(viewport->buffer.src_width) != -1)
> +               return false;
> +       if (viewport->surface.width != -1)
> +               return false;


Presumably one of these should be height, and are we not missing a
src_height check too? Also, might as well merge the buffer_scale check in
here too.

Cheers,
Daniel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160930/d0edf673/attachment-0001.html>


More information about the wayland-devel mailing list