[PATCH weston 2/2] Fix build warnings from image_loader
Kristian Høgsberg
hoegsberg at gmail.com
Fri Nov 30 11:17:21 PST 2012
On Fri, Nov 30, 2012 at 01:37:28PM +0200, Pekka Paalanen wrote:
> Fix the following build warnings, and the build failures due to the
> warning fixes:
>
> CC libshared_cairo_la-image-loader.lo
> image-loader.c:369:1: warning: no previous prototype for 'load_image'
>
> CC x11_backend_la-compositor-x11.lo
> compositor-x11.c: In function 'x11_output_set_icon':
> compositor-x11.c:396:2: warning: implicit declaration of function 'load_image'
> compositor-x11.c:396:8: warning: assignment makes pointer from integer without a cast
>
> CC wayland_backend_la-compositor-wayland.lo
> compositor-wayland.c: In function 'create_border':
> compositor-wayland.c:97:2: warning: implicit declaration of function 'load_image'
> compositor-wayland.c:97:8: warning: assignment makes pointer from integer without a cast
Oops, yes, sorry about that.
Kristian
> Signed-off-by: Pekka Paalanen <ppaalanen at gmail.com>
> ---
> shared/cairo-util.h | 1 +
> shared/image-loader.c | 2 +-
> src/Makefile.am | 4 ++++
> src/compositor-wayland.c | 1 +
> src/compositor-x11.c | 1 +
> src/xwayland/Makefile.am | 6 +++++-
> 6 files changed, 13 insertions(+), 2 deletions(-)
>
> diff --git a/shared/cairo-util.h b/shared/cairo-util.h
> index 64ed03f..3b93d40 100644
> --- a/shared/cairo-util.h
> +++ b/shared/cairo-util.h
> @@ -23,6 +23,7 @@
> #ifndef _CAIRO_UTIL_H
> #define _CAIRO_UTIL_H
>
> +#include <cairo.h>
> #include <pixman.h>
>
> void
> diff --git a/shared/image-loader.c b/shared/image-loader.c
> index 1960bb5..64ba2ae 100644
> --- a/shared/image-loader.c
> +++ b/shared/image-loader.c
> @@ -29,7 +29,7 @@
> #include <png.h>
> #include <pixman.h>
>
> -#include "config-parser.h"
> +#include "cairo-util.h"
>
> #define ARRAY_LENGTH(a) (sizeof (a) / sizeof (a)[0])
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 0d24084..e8315ca 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -96,6 +96,8 @@ x11_backend_la_LIBADD = $(COMPOSITOR_LIBS) $(X11_COMPOSITOR_LIBS) \
> ../shared/libshared-cairo.la
> x11_backend_la_CFLAGS = \
> $(COMPOSITOR_CFLAGS) \
> + $(PIXMAN_CFLAGS) \
> + $(CAIRO_CFLAGS) \
> $(X11_COMPOSITOR_CFLAGS) \
> $(GCC_CFLAGS)
> x11_backend_la_SOURCES = compositor-x11.c
> @@ -129,6 +131,8 @@ wayland_backend_la_LIBADD = $(COMPOSITOR_LIBS) $(WAYLAND_COMPOSITOR_LIBS) \
> ../shared/libshared-cairo.la
> wayland_backend_la_CFLAGS = \
> $(COMPOSITOR_CFLAGS) \
> + $(PIXMAN_CFLAGS) \
> + $(CAIRO_CFLAGS) \
> $(WAYLAND_COMPOSITOR_CFLAGS) \
> $(GCC_CFLAGS)
> wayland_backend_la_SOURCES = compositor-wayland.c
> diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
> index 24dccd1..167fce6 100644
> --- a/src/compositor-wayland.c
> +++ b/src/compositor-wayland.c
> @@ -38,6 +38,7 @@
>
> #include "compositor.h"
> #include "gl-renderer.h"
> +#include "../shared/cairo-util.h"
>
> struct wayland_compositor {
> struct weston_compositor base;
> diff --git a/src/compositor-x11.c b/src/compositor-x11.c
> index b26da09..04a1803 100644
> --- a/src/compositor-x11.c
> +++ b/src/compositor-x11.c
> @@ -48,6 +48,7 @@
> #include "compositor.h"
> #include "gl-renderer.h"
> #include "../shared/config-parser.h"
> +#include "../shared/cairo-util.h"
>
> #define DEFAULT_AXIS_STEP_DISTANCE wl_fixed_from_int(10)
>
> diff --git a/src/xwayland/Makefile.am b/src/xwayland/Makefile.am
> index 1c3b114..8f3bddd 100644
> --- a/src/xwayland/Makefile.am
> +++ b/src/xwayland/Makefile.am
> @@ -12,7 +12,11 @@ xwayland_la_LDFLAGS = -module -avoid-version
> xwayland_la_LIBADD = \
> $(XWAYLAND_LIBS) \
> $(top_builddir)/shared/libshared-cairo.la
> -xwayland_la_CFLAGS = $(GCC_CFLAGS) $(COMPOSITOR_CFLAGS)
> +xwayland_la_CFLAGS = \
> + $(GCC_CFLAGS) \
> + $(COMPOSITOR_CFLAGS) \
> + $(PIXMAN_CFLAGS) \
> + $(CAIRO_CFLAGS)
> xwayland_la_SOURCES = \
> xwayland.h \
> window-manager.c \
> --
> 1.7.8.6
>
More information about the wayland-devel
mailing list