On 16/11/16 08:49 AM, Daniel Stone wrote:
Hi,

On 30 September 2016 at 17:13, Derek Foreman <[email protected]> wrote:
If only the source of a viewport is set, the width and height must
be integer or the protocol mandates that the compositor generate an
error.  This is because using only the source is a crop, and the
width and height become the surface size - all surface sizes must
be integer.

Weston was fixed to generate this error in bb32ccc0, however the
test app continued to use fractional co-ordinates when run as
weston-scaler -s  (which only sets the viewport source)

This leaves fractional width/height for the other cases, but uses
integer for the crop-only mode.  The descriptions in the help text
are still accurate with this change, but weston-scaler -s no longer
exits with an error.

Signed-off-by: Derek Foreman <[email protected]>

Pushed with review, thanks. I'd like to just subsume the cursor-plane
one into the general plane_state rework anyway, if that's OK with you?

Absolutely!

To ssh://git.freedesktop.org/git/wayland/weston
   be8a6d3..df84dbe  push -> master

Thanks,
Derek

Cheers,
Daniel


_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to