[PATCH wayland] introduce new headers wayland-client/server-core.h
spitzak at gmail.com
Thu Apr 30 10:41:19 PDT 2015
On 04/30/2015 03:34 AM, Pekka Paalanen wrote:
>> Just to be clear, my proposal is:
>> wayland-client.h : hand-written, includes:
>> deprecated api
>> wayland-client-core.h : hand-written, includes
>> wl_event_queue, wl_proxy, etc
>> *-protocol.h : generated, includes
>> inline wrappers for messages and events
> Once again, after discussion, you ended up proposing exactly what the
> original poster's original patches did.
Yes I think you are right, this is the *original* proposal. It is
however not what is in the current patches.
I am trying to remove the circular dependency in the current headers and
in the "!include_core_only" default case for wayland-scanner. It would
also be nice to not force programs using a protocol to include a
Therefore I believe the original poster's version is the correct solution.
If you really want to be anal about back-compatibility it could be
solved this way, with not
If back compatibility is really a concern, a better fix would be this:
wayland-client.h : deprecated, hand-written
wayland-client-core.h : hand-written, matches existing proposal
*-protocol-client.h : generated by wayland-scanner
wrappers for messages/events/proxy
*-client-protocol.h : very short file generated by wayland-scanner
comment that this file is deprecated
IMHO back-compatibility is NOT a problem. I posted a patch for weston
and it was 2 lines.
More information about the wayland-devel