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

Bryce W. Harrington b.harrington at samsung.com
Tue Mar 18 12:34:56 PDT 2014


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(-)

diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
index 238946b..83eb59c 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\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\n", height * stride);
 		close(fd);
 		return NULL;
 	}
-- 
1.7.9.5


More information about the wayland-devel mailing list