Re: [PATCH wayland] tests: add an headers test

2015-05-21 Thread Pekka Paalanen
On Wed, 20 May 2015 15:47:00 -0700 Bill Spitzak wrote: > Please don't duplicate the functions in multiple headers! > > You could achieve the same result this way so that there is only one > copy of each function in the headers: > > wayland-client-protocol-core.h = >(the same as you have it

Re: [PATCH wayland] tests: add an headers test

2015-05-21 Thread Pekka Paalanen
On Wed, 20 May 2015 22:38:53 +0300 Giulio Camuffo wrote: > This test checks that the protocol and library headers include only what > they are supposed to include. That is, that the core headers do not > include the protocol headers and that the core protocol headers do not > include the non core

Re: [PATCH wayland] tests: add an headers test

2015-05-20 Thread Bill Spitzak
Please don't duplicate the functions in multiple headers! You could achieve the same result this way so that there is only one copy of each function in the headers: wayland-client-protocol-core.h = (the same as you have it, generated by wayland-scanner -c) wayland-client-protocol.h = #inc

[PATCH wayland] tests: add an headers test

2015-05-20 Thread Giulio Camuffo
This test checks that the protocol and library headers include only what they are supposed to include. That is, that the core headers do not include the protocol headers and that the core protocol headers do not include the non core library headers. The build process now generates core protocol hea