[PATCH wayland v3 0/4] Enum and bitfield XML attributes

[Pekka Paalanen]

On Wed, 4 Nov 2015 15:03:12 -0800
Bryce Harrington <bryce at osg.samsung.com> wrote:

> On Sat, Oct 24, 2015 at 12:07:46PM +0100, Auke Booij wrote:
> > There has been plenty of discussion regarding the introduction of new XML
> > attributes. This series of patches improves on my earlier attempt to find
> > common ground in this.
> > 
> > Major exclusions from these patches are:
> > 
> >  - Support for cross-interface enum referencing (e.g.
> >    wl_shm_pool::create_buffer::format to wl_shm::format, and
> >    wl_surface::set_buffer_transform::transform to wl_output::transform).
> >    There is consensus that this should be added sooner rather than later.
> >    My proposal is to get these patches in first, and then look at that.
> > 
> >  - Open/closed enum specification. There is still too much debate on this.
> >    Until there is any firm specification of open/closed enums, every enum
> >    should be considered open, and although some legal values might be
> >    listed, others might not be, and some might only be legal sometimes.
> >    New values may be added (but not changed or removed) to protocol
> >    specifications without introducing any compatibility issues.
> > 
> > Changes since v2:
> >  - Incorporate new documentation drawn up during v2 discussion
> >  - Enable wl_shell_surface::transform bitfield
> > 
> > Changes since v1:
> >  - Split up documentation patches
> >  - Set enum="fullscreen_method" on
> >    wl_shell_surface::set_fullscreen::method
> > 
> > Auke Booij (4):
> >   doc: document the enum and bitfield attributes
> >   protocol: specify enum and bitfield attributes
> >   scanner: enforce correct argument type for enums
> >   doc: output enum and bitfield attributes in the documentation
> > 
> >  doc/publican/protocol-to-docbook.xsl |  9 +++++
> >  doc/publican/sources/Protocol.xml    | 41 +++++++++++++++++----
> >  protocol/wayland.xml                 | 36 +++++++++----------
> >  src/scanner.c                        | 70 ++++++++++++++++++++++++++++++++++++
> >  4 files changed, 132 insertions(+), 24 deletions(-)
> 
> Thanks, was hoping to get another r-b or two but guess we've adequately
> exhausted interest in this particular changeset.  Let's get this landed
> so y'all can proceed with the next step in the enum work.
> 
> remote: Updating patchwork state for http://patchwork.freedesktop.org/project/wayland/list/
> remote: I: patch #62885 updated using rev 999225c17a6ee23d5b5171b80b5158f93beb7abc
> remote: I: patch #62835 updated using rev 851614fa78862499e016c5718e730fefbb8e3b73
> remote: E: failed to find patch for rev 1771299a5a965c1f95c18ef795c29db876133a81
> remote: I: patch #62244 updated using rev 4a97cb35794866414b84242a89d1dbcc7f9c59fe
> remote: I: 3 patch(es) updated to state Accepted.
> To ssh://git.freedesktop.org/git/wayland/wayland
>    f150d7a..4a97cb3  master -> master

Hi all,

good work everyone!

I've been catching up after my sick leave, and it seems things turned
out a way I am happy with so far. :-)


Thanks,
pq
-------------- 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/d1b21e09/attachment.sig>