<div dir="ltr">This is just some small FYI status feedback from the Java bindings, so it doesn't get lost/people know.<div><br></div><div><br></div><div>from irc #wayland:</div><div><br></div><div><div>19:57 < bryce> anyone working on or interested in language bindings, would be great to get your Reviewed-by or even just Acked-by on Auke's enum patchset from today.</div><div>19:58 < bryce> We're about ready to land it, but since it's a change to the protocol syntax want to just check that we've had a wide enough array of folks take a look at it.</div><div><br></div><div>19:59 < zubzub> if I understand correctly they don't implement open/closed enums yet right?</div><div>19:59 < zubzub> and they don't guarentee order right?<br></div><div>20:02 < zubzub> and the order thing will probably be never guaranteed as I understand it(?)</div><div>20:02 < zubzub> at least it's not formally agreed like that aside from "be compatible" whatever that means</div></div><div><br></div><div>20:19 < bryce> closed enums are left for future work<br></div><div>20:20 < bryce> I'd need to doublecheck on the ordering implications<br></div><div><br></div><div>21:00 < zubzub> so yeah, as lang as the 2 things I mentioned arent resolved, the whole enum thing will not be applicable for me (Java)<br></div><div><br></div><div>21:08 < bryce> zubzub, mind commenting so on the list?<br></div><div><br></div><div>21:17 < zubzub> bryce: sure I'll c/p the irc convo for record keeping</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 24, 2015 at 1:07 PM, Auke Booij <span dir="ltr"><<a href="mailto:auke@tulcod.com" target="_blank">auke@tulcod.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Auke Booij <<a href="mailto:auke@tulcod.com">auke@tulcod.com</a>><br>
---<br>
 protocol/wayland.xml | 36 ++++++++++++++++++------------------<br>
 1 file changed, 18 insertions(+), 18 deletions(-)<br>
<br>
diff --git a/protocol/wayland.xml b/protocol/wayland.xml<br>
index 59819e9..9c22d45 100644<br>
--- a/protocol/wayland.xml<br>
+++ b/protocol/wayland.xml<br>
@@ -367,7 +367,7 @@<br>
        can be used for buffers. Known formats include<br>
        argb8888 and xrgb8888.<br>
       </description><br>
-      <arg name="format" type="uint"/><br>
+      <arg name="format" type="uint" enum="format"/><br>
     </event><br>
   </interface><br>
<br>
@@ -746,7 +746,7 @@<br>
       <arg name="serial" type="uint" summary="serial of the implicit grab on the pointer"/><br>
     </request><br>
<br>
-    <enum name="resize"><br>
+    <enum name="resize" bitfield="true"><br>
       <description summary="edge values for resizing"><br>
        These values are used to indicate which edge of a surface<br>
        is being dragged in a resize operation. The server may<br>
@@ -774,7 +774,7 @@<br>
       </description><br>
       <arg name="seat" type="object" interface="wl_seat" summary="the wl_seat whose pointer is used"/><br>
       <arg name="serial" type="uint" summary="serial of the implicit grab on the pointer"/><br>
-      <arg name="edges" type="uint" summary="which edge or corner is being dragged"/><br>
+      <arg name="edges" type="uint" enum="resize" summary="which edge or corner is being dragged"/><br>
     </request><br>
<br>
     <request name="set_toplevel"><br>
@@ -785,7 +785,7 @@<br>
       </description><br>
     </request><br>
<br>
-    <enum name="transient"><br>
+    <enum name="transient" bitfield="true"><br>
       <description summary="details of transient behaviour"><br>
        These flags specify details of the expected behaviour<br>
        of transient surfaces. Used in the set_transient request.<br>
@@ -807,7 +807,7 @@<br>
       <arg name="parent" type="object" interface="wl_surface"/><br>
       <arg name="x" type="int"/><br>
       <arg name="y" type="int"/><br>
-      <arg name="flags" type="uint"/><br>
+      <arg name="flags" type="uint" enum="transient"/><br>
     </request><br>
<br>
     <enum name="fullscreen_method"><br>
@@ -858,7 +858,7 @@<br>
        with the dimensions for the output on which the surface will<br>
        be made fullscreen.<br>
       </description><br>
-      <arg name="method" type="uint"/><br>
+      <arg name="method" type="uint" enum="fullscreen_method"/><br>
       <arg name="framerate" type="uint"/><br>
       <arg name="output" type="object" interface="wl_output" allow-null="true"/><br>
     </request><br>
@@ -891,7 +891,7 @@<br>
       <arg name="parent" type="object" interface="wl_surface"/><br>
       <arg name="x" type="int"/><br>
       <arg name="y" type="int"/><br>
-      <arg name="flags" type="uint"/><br>
+      <arg name="flags" type="uint" enum="transient"/><br>
     </request><br>
<br>
     <request name="set_maximized"><br>
@@ -972,7 +972,7 @@<br>
        in surface local coordinates.<br>
       </description><br>
<br>
-      <arg name="edges" type="uint"/><br>
+      <arg name="edges" type="uint" enum="resize"/><br>
       <arg name="width" type="int"/><br>
       <arg name="height" type="int"/><br>
     </event><br>
@@ -1337,7 +1337,7 @@<br>
       maintains a keyboard focus and a pointer focus.<br>
     </description><br>
<br>
-    <enum name="capability"><br>
+    <enum name="capability" bitfield="true"><br>
       <description summary="seat capability bitmask"><br>
         This is a bitmask of capabilities this seat has; if a member is<br>
         set, then it is present on the seat.<br>
@@ -1353,7 +1353,7 @@<br>
        keyboard or touch capabilities.  The argument is a capability<br>
        enum containing the complete set of capabilities this seat has.<br>
       </description><br>
-      <arg name="capabilities" type="uint"/><br>
+      <arg name="capabilities" type="uint" enum="capability"/><br>
     </event><br>
<br>
     <request name="get_pointer"><br>
@@ -1530,7 +1530,7 @@<br>
       <arg name="serial" type="uint"/><br>
       <arg name="time" type="uint" summary="timestamp with millisecond granularity"/><br>
       <arg name="button" type="uint"/><br>
-      <arg name="state" type="uint"/><br>
+      <arg name="state" type="uint" enum="button_state"/><br>
     </event><br>
<br>
     <enum name="axis"><br>
@@ -1562,7 +1562,7 @@<br>
       </description><br>
<br>
       <arg name="time" type="uint" summary="timestamp with millisecond granularity"/><br>
-      <arg name="axis" type="uint"/><br>
+      <arg name="axis" type="uint" enum="axis"/><br>
       <arg name="value" type="fixed"/><br>
     </event><br>
<br>
@@ -1602,7 +1602,7 @@<br>
        This event provides a file descriptor to the client which can be<br>
        memory-mapped to provide a keyboard mapping description.<br>
       </description><br>
-      <arg name="format" type="uint"/><br>
+      <arg name="format" type="uint" enum="keymap_format"/><br>
       <arg name="fd" type="fd"/><br>
       <arg name="size" type="uint"/><br>
     </event><br>
@@ -1647,7 +1647,7 @@<br>
       <arg name="serial" type="uint"/><br>
       <arg name="time" type="uint" summary="timestamp with millisecond granularity"/><br>
       <arg name="key" type="uint"/><br>
-      <arg name="state" type="uint"/><br>
+      <arg name="state" type="uint" enum="key_state"/><br>
     </event><br>
<br>
     <event name="modifiers"><br>
@@ -1828,17 +1828,17 @@<br>
           summary="width in millimeters of the output"/><br>
       <arg name="physical_height" type="int"<br>
           summary="height in millimeters of the output"/><br>
-      <arg name="subpixel" type="int"<br>
+      <arg name="subpixel" type="int" enum="subpixel"<br>
           summary="subpixel orientation of the output"/><br>
       <arg name="make" type="string"<br>
           summary="textual description of the manufacturer"/><br>
       <arg name="model" type="string"<br>
           summary="textual description of the model"/><br>
-      <arg name="transform" type="int"<br>
+      <arg name="transform" type="int" enum="transform"<br>
           summary="transform that maps framebuffer to output"/><br>
     </event><br>
<br>
-    <enum name="mode"><br>
+    <enum name="mode" bitfield="true"><br>
       <description summary="mode information"><br>
        These flags describe properties of an output mode.<br>
        They are used in the flags bitfield of the mode event.<br>
@@ -1865,7 +1865,7 @@<br>
         the output may be scaled, as described in wl_output.scale,<br>
         or transformed , as described in wl_output.transform.<br>
       </description><br>
-      <arg name="flags" type="uint" summary="bitfield of mode flags"/><br>
+      <arg name="flags" type="uint" enum="mode" summary="bitfield of mode flags"/><br>
       <arg name="width" type="int" summary="width of the mode in hardware units"/><br>
       <arg name="height" type="int" summary="height of the mode in hardware units"/><br>
       <arg name="refresh" type="int" summary="vertical refresh rate in mHz"/><br>
<span class="HOEnZb"><font color="#888888">--<br>
2.6.1<br>
<br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</font></span></blockquote></div><br></div>