[PATCH wayland 2/3] protocol: allow for a <description> element below <protocol>

Pekka Paalanen ppaalanen at gmail.com
Mon Nov 9 01:33:22 PST 2015


On Mon,  9 Nov 2015 14:15:00 +1000
Peter Hutterer <peter.hutterer at who-t.net> wrote:

> The scanner parses this already, it doesn't do anything with it though.
> 
> The DTD requires the order to be copyright, description, then the interfaces.
> That's largely a DTD limitation, the scanner doesn't care.

Hi Peter,

this is slightly unfortunate but I suppose harmless, especially when
the DTD check is non-fatal. Very good to point it out.

Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Looks like we will need another patch to add the new 'enum' and
'bitfield' attributes, those produce warnings at the moment.


Thanks,
pq


> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>  protocol/wayland.dtd | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/protocol/wayland.dtd b/protocol/wayland.dtd
> index b8b1573..e28dbc0 100644
> --- a/protocol/wayland.dtd
> +++ b/protocol/wayland.dtd
> @@ -1,4 +1,4 @@
> -<!ELEMENT protocol (copyright?, interface+)>
> +<!ELEMENT protocol (copyright?, description?, interface+)>
>    <!ATTLIST protocol name CDATA #REQUIRED>
>  <!ELEMENT copyright (#PCDATA)>
>  <!ELEMENT interface (description?,(request|event|enum)+)>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20151109/e96b6f0d/attachment.sig>


More information about the wayland-devel mailing list