[PATCH wayland] introduce new headers wayland-client/server-core.h
mchqwerty at gmail.com
Mon Apr 27 23:29:59 PDT 2015
On Mon, Apr 27, 2015 at 4:06 PM, Giulio Camuffo <giuliocamuffo at gmail.com>
> 2015-04-27 21:53 GMT+03:00 Bill Spitzak <spitzak at gmail.com>:
> > On 04/26/2015 11:48 PM, Marek Chalupa wrote:
> >> Is the --include-core-headers option necessary? Until now the scanner
> >> included wayland-client/server.h
> >> which included the %-protocol.h, but since there are inclusion guards,
> >> the protocol header was not
> >> included again - so including wayland-client/server.h is the same as
> >> including wayland-client/server-core.h, isn't it?
> >> So including the core headers should be good for all cases.
> The problem is when you have an extension.xml, and generate
> extension-client-protocol.h. Then you have some preexisting code
> including that and relying on it including wayland-client.h and then
> wayland-client-protocol.h, and if you use something from that it will
> not build anymore, because wayland-client-core.h doesn't include it.
So when you want to include only core file and not whole
wayland-client/server.h in generated headers?
> > Even if there is a reason to not include the core headers (I don't see
> > however) it certainly would be the default to include them. Therefore the
> > switch should at least be reversed so no switch means to include them.
> No, because it would then not be backward compatible, breaking the
> sole reason the switch exists for.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the wayland-devel