<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 5, 2014 at 7:51 AM, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br> <br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
</span>I took a look at both of them, you are making an incompatible change to<br>
wayland.xml, and breaking the generator.<br>
<br>
Essentially you are breaking the unwritten specification on how the XML<br>
files are processed into wire format and language bindings.<br>
Wayland-scanner is not the only generator, and wayland.xml is not the<br>
only XML file in existence.<br>
<span class=""></span><span class="im"><br>
</span>If you look carefully, you see that there is absolutely no<br>
documentation on how to generate the wire protocol or C function<br>
signatures from the XML. It is all undocumented.</blockquote></div><br></div><div class="gmail_extra">Thanks for the continued dialog.  I was also surprised that the reqest / event opcodes were not specified in the specification but was mentioned in the bug report for improved documentation.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">The one major critique that you pointed out in your previous message is that this patch is completely non-generic and written under the (apparently false) assumption that the scanner is only used to provide the code-gen for the wayland.xml provided in the wayland repo.  The intention was to possibly elicit help to replace the string comparisons with something more generic.<br><br></div><div class="gmail_extra">If indeed the scanner application is used by external projects or to parse external specifications (as you'd mentioned custom extensions) then indeed this changeset is not generic enough for that case.  I think that's probably the main factor to why the unique rule is needed, because I don't think it's possible to come to a generic solution without also changing the api.<br><br></div><div class="gmail_extra">At least now there's some dialog in the mailing list for others to consider until the docs get updated.  Again I thank you for taking the time (and patience) to help me get up to speed.<br><br></div><div class="gmail_extra">Paul<br></div></div>