[PATCH wayland] introduce new headers wayland-client/server-core.h

[Bill Spitzak]

On 04/27/2015 01:06 PM, Giulio Camuffo wrote:
> 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.
>
>>
>>
>> Even if there is a reason to not include the core headers (I don't see it,
>> 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.

I think there is some confusion here.

I am saying the default should be to *include* the headers, so that 
existing code will continue to work.

What I (and I believe others) are trying to say is that this should not 
be an option, it *ALWAYS* includes the core headers. There is no reason 
for a switch to turn it off, and certainly no reason for a switch that 
is required to turn it on.