O<span></span>n Thursday, March 19, 2015, Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 03/19/2015 07:07 AM, Jonny Lamb wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-       if (gl_renderer->output_create(&<u></u>output->base, output->surface,<br>
+       if (gl_renderer->output_create(&<u></u>output->base,<br>
+                                      output->surface, output->surface,<br>
                                       gl_renderer->opaque_attribs,<br>
                                       &format) < 0) {<br>
</blockquote>
<br>
I don't see the need for the duplicate arguments here. Both are obviously the same value (the xid), it's just that one of the API's declares this as EGLNativeWindowType and the other as void*. I think it would be best to hide all this ugliness and make output_create take an xid.<br>
</blockquote><div><br></div><div>Keep reading through the diff until you see compositor-x11.c, which has a comment explaining why it's necessary. Passing an XID directly doesn't help at all. </div>