[PATCH xserver] xwayland: replace hardcoded function name with __func__ in error msg
Olivier Fourdan
ofourdan at redhat.com
Tue Feb 7 08:11:35 UTC 2017
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> hw/xwayland/xwayland-input.c | 4 ++--
> hw/xwayland/xwayland-output.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
> index 580df09..8d8bcbd 100644
> --- a/hw/xwayland/xwayland-input.c
> +++ b/hw/xwayland/xwayland-input.c
> @@ -854,7 +854,7 @@ touch_handle_down(void *data, struct wl_touch *wl_touch,
>
> xwl_touch = calloc(1, sizeof *xwl_touch);
> if (xwl_touch == NULL) {
> - ErrorF("touch_handle_down ENOMEM");
> + ErrorF("%s ENOMEM", __func__);
> return;
> }
>
> @@ -1125,7 +1125,7 @@ create_input_device(struct xwl_screen *xwl_screen,
> uint32_t id, uint32_t version
>
> xwl_seat = calloc(1, sizeof *xwl_seat);
> if (xwl_seat == NULL) {
> - ErrorF("create_input ENOMEM\n");
> + ErrorF("%s ENOMEM", __func__);
> return;
> }
>
> diff --git a/hw/xwayland/xwayland-output.c b/hw/xwayland/xwayland-output.c
> index f3ce763..17715e2 100644
> --- a/hw/xwayland/xwayland-output.c
> +++ b/hw/xwayland/xwayland-output.c
> @@ -244,7 +244,7 @@ xwl_output_create(struct xwl_screen *xwl_screen, uint32_t
> id)
>
> xwl_output = calloc(1, sizeof *xwl_output);
> if (xwl_output == NULL) {
> - ErrorF("create_output ENOMEM\n");
> + ErrorF("%s ENOMEM", __func__);
> return NULL;
> }
>
> --
> 2.9.3
LGTM - Yo get rid of two \n along the way, but I think there were not needed in the first place so:
Reviewed-by: Olivier Fourdan <ofourdan at redhat.com>
More information about the xorg-devel
mailing list