[PATCH v5 xdg-shell-unstable-v6] xdg-shell: Add min/max size requests

Mike Blumenkrantz michael.blumenkrantz at gmail.com
Tue Apr 12 15:11:00 UTC 2016


Okay, if we're not going with uints then at the least can the "use 0 to
unset min/max" be changed to "use <= 0 to unset min/max" to explicitly
cover that case?

On Tue, Apr 12, 2016 at 10:40 AM Olivier Fourdan <ofourdan at redhat.com>
wrote:

> Hi,
>
>
> > > Good point. Setting an invalid state should probably result in a
> > > protocol error.
> >
> > No this cannot be a protocol error because that makes it difficult to
> > change the size range to a new one that does not intersect the old one.
> >
> > Perhaps having the commit of an invalid state be a protocol error is ok.
>
> We'd reach an invalid state only once it's applied, ie after the commit.
>
> So the client can (and must) set a Min <= Max before the commit, otherwise
> it's an invalid state that will lead to a protocol error.
>
> That's what I've tried to specify in v6:
>
>         Values set in this way are double-buffered. They will get applied
>         on the next commit.
>
>         [...]
>
>         Requesting a minimum size to be larger than the maximum size of
>         a surface is illegal and will result in a protocol error.
>
> Cheers,
> Olivier
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160412/21a2e421/attachment.html>


More information about the wayland-devel mailing list