Le 22/03/2014 12:21, Pekka Paalanen a écrit :
On Fri, 21 Mar 2014 05:54:02 +0000
"Bryce W. Harrington" <[email protected]> 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(-)
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
Looks good!
Good for me too, thanks.
--
David FORT
website: http://www.hardening-consulting.com/
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
