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 <[email protected]>
> ---
> 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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
