<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo <span dir="ltr"><<a href="mailto:giuliocamuffo@gmail.com" target="_blank">giuliocamuffo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">2015-04-27 21:53 GMT+03:00 Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>>:<br>
> On 04/26/2015 11:48 PM, Marek Chalupa wrote:<br>
><br>
>><br>
>> Is the --include-core-headers option necessary? Until now the scanner<br>
>> included wayland-client/server.h<br>
>> which included the %-protocol.h, but since there are inclusion guards,<br>
>> the protocol header was not<br>
>> included again - so including wayland-client/server.h is the same as<br>
>> including wayland-client/server-core.h, isn't it?<br>
>> So including the core headers should be good for all cases.<br>
<br>
</span>The problem is when you have an extension.xml, and generate<br>
extension-client-protocol.h. Then you have some preexisting code<br>
including that and relying on it including wayland-client.h and then<br>
wayland-client-protocol.h, and if you use something from that it will<br>
not build anymore, because wayland-client-core.h doesn't include it.<br>
<span class=""><br></span></blockquote><div><br></div><div>True...<br>So when you want to include only core file and not whole wayland-client/server.h in generated headers?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
><br>
> Even if there is a reason to not include the core headers (I don't see it,<br>
> however) it certainly would be the default to include them. Therefore the<br>
> switch should at least be reversed so no switch means to include them.<br>
<br>
</span>No, because it would then not be backward compatible, breaking the<br>
sole reason the switch exists for.<br></blockquote><div><br></div><div>Thanks,<br></div><div>Marek <br></div></div><br></div></div>