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 Bryce _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel
