[PATCH] compositor: Use weston_log rather than perror for error messages

[Kristian Høgsberg]

On Fri, Mar 21, 2014 at 05:54:02AM +0000, Bryce W. Harrington wrote:
> weston_log() seems to be the standard elsewhere in the codebase for
> errors.  These are the only two instances where perror() is used
> instead, and their error messages aren't that informative anyway.
> 
> Signed-off-by: Bryce Harrington <b.harrington at samsung.com>
> ---
>  src/compositor-wayland.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Grabbed this version instead, thanks.

Kristian

> diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
> index 238946b..0f67419 100644
> --- a/src/compositor-wayland.c
> +++ b/src/compositor-wayland.c
> @@ -211,13 +211,13 @@ wayland_output_get_shm_buffer(struct wayland_output *output)
>  
>  	fd = os_create_anonymous_file(height * stride);
>  	if (fd < 0) {
> -		perror("os_create_anonymous_file");
> +		weston_log("could not create an anonymous file buffer: %m\n");
>  		return NULL;
>  	}
>  
>  	data = mmap(NULL, height * stride, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
>  	if (data == MAP_FAILED) {
> -		perror("mmap");
> +		weston_log("could not mmap %d memory for data: %m\n", height * stride);
>  		close(fd);
>  		return NULL;
>  	}
> -- 
> 1.7.9.5
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel