<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 1, 2015 at 12:09 PM, Nils Chr. Brause <span dir="ltr"><<a href="mailto:nilschrbrause@gmail.com" target="_blank">nilschrbrause@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Oct 1, 2015 at 9:00 PM, Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Oct 1, 2015 at 8:09 AM, Nils Chr. Brause <<a href="mailto:nilschrbrause@gmail.com">nilschrbrause@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> I would recommend to also include the name of the interface, where the<br>
>> enum is defined, in the enum attribute, e.g:<br>
>> <arg name="format" type="uint" enum="wl_shm.format"/><br>
><br>
><br>
> I believe popular opinion was that a "local" enum, belonging to the same<br>
> object as the message or event, should not require this prefix. One big<br>
> reason is so that the object can be renamed without having to edit it's<br>
> contents.<br>
><br>
> And currently only such "local" enums are being used in the protocols, so<br>
> there is no need for parsers to support any such prefix yet, and no need to<br>
> figure out it's rules. Whether the ability to get enums from other objects,<br>
> or from some global space, does not have to be decided yet.<br>
<br>
There are "non-local" enums, e.g.<br>
wl_surface::set_buffer_transform::transform uses the enum<br>
wl_output::transform, or am I missing something important here?<br></blockquote><div><br></div><div>It looks like you are right. Not sure what was done, I have not looked at it for a long time now. However whatever is in the patch seems to work and I would use that.<br><br></div></div><br></div></div>