[PATCH wayland] event-loop: Make transitive include explicit

Yong Bakos junk at humanoriented.com
Thu May 19 16:39:13 UTC 2016


On May 17, 2016, at 2:32 AM, Jonas Ådahl <jadahl at gmail.com> wrote:
> 
> On Mon, May 16, 2016 at 12:05:39PM -0600, Yong Bakos wrote:
>> From: Yong Bakos <ybakos at humanoriented.com>
>> 
>> The explicit inclusion of wayland-server.h hides the real dependency, which
>> is wayland-server-core.h.
>> 
>> Signed-off-by: Yong Bakos <ybakos at humanoriented.com>
> 
> Reviewed-by: Jonas Ådahl <jadahl at gmail.com>

Thanks for the review. But to be sure, this change wouldn't affect
backward-compatibility, would it?

yong


>> ---
>> src/event-loop.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/src/event-loop.c b/src/event-loop.c
>> index ea27b69..11a9bf2 100644
>> --- a/src/event-loop.c
>> +++ b/src/event-loop.c
>> @@ -37,7 +37,7 @@
>> #include <sys/timerfd.h>
>> #include <unistd.h>
>> #include "wayland-private.h"
>> -#include "wayland-server.h"
>> +#include "wayland-server-core.h"
>> #include "wayland-os.h"
>> 
>> struct wl_event_loop {
>> -- 
>> 2.7.2
>> 
>> _______________________________________________
>> wayland-devel mailing list
>> wayland-devel at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/wayland-devel



More information about the wayland-devel mailing list