[PATCH wayland] protocol: Add a surface_has_existing_role error

Jason Ekstrand jason at jlekstrand.net
Fri Aug 22 10:48:02 PDT 2014


As I mentioned on IRC, I don't really like adding and using an enum value
without bumping protocol version.  However, the only time we ever use it is
to kill the client so the worst thing that can happen is that the client
doesn't report the error correctly.  I think I'm ok with it.
--Jason Ekstrand


On Fri, Aug 22, 2014 at 10:37 AM, Jasper St. Pierre <jstpierre at mecheye.net>
wrote:

> This will be used by implementations to send out errors if clients try
> to set roles on surfaces that already have roles.
> ---
>  protocol/wayland.xml | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/protocol/wayland.xml b/protocol/wayland.xml
> index bb457bc..f6eb54d 100644
> --- a/protocol/wayland.xml
> +++ b/protocol/wayland.xml
> @@ -983,6 +983,7 @@
>        </description>
>        <entry name="invalid_scale" value="0" summary="buffer scale value
> is invalid"/>
>        <entry name="invalid_transform" value="1" summary="buffer transform
> value is invalid"/>
> +      <entry name="surface_has_existing_role" value="2" summary="the
> surface passed already has a role"/>
>      </enum>
>
>      <request name="destroy" type="destructor">
> --
> 2.1.0
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140822/06fb4b0d/attachment.html>


More information about the wayland-devel mailing list