<div dir="ltr">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?</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 12, 2016 at 10:40 AM Olivier Fourdan <<a href="mailto:ofourdan@redhat.com">ofourdan@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
<br>
> > Good point. Setting an invalid state should probably result in a<br>
> > protocol error.<br>
><br>
> No this cannot be a protocol error because that makes it difficult to<br>
> change the size range to a new one that does not intersect the old one.<br>
><br>
> Perhaps having the commit of an invalid state be a protocol error is ok.<br>
<br>
We'd reach an invalid state only once it's applied, ie after the commit.<br>
<br>
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.<br>
<br>
That's what I've tried to specify in v6:<br>
<br>
        Values set in this way are double-buffered. They will get applied<br>
        on the next commit.<br>
<br>
        [...]<br>
<br>
        Requesting a minimum size to be larger than the maximum size of<br>
        a surface is illegal and will result in a protocol error.<br>
<br>
Cheers,<br>
Olivier<br>
</blockquote></div>