ecore_evas: correct error message reporting incorrect engine name
authorBryce Harrington <bryce@osg.samsung.com>
Mon, 5 Jun 2017 18:05:53 +0000 (11:05 -0700)
committerCedric BAIL <cedric@osg.samsung.com>
Mon, 5 Jun 2017 19:07:56 +0000 (12:07 -0700)
Summary:
_ecore_evas_wl_common_new_internal() creates both wayland_egl and
wayland_shm backed windows, so reporting that the failure was in looking
up "Wayland_Shm" could be misleading.

Also, this routine can be called with any arbitrary string as
engine_name, so including what was received in the error message might
be helpful for diagnosing bugs.

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

Differential Revision: https://phab.enlightenment.org/D4897

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/modules/ecore_evas/engines/wayland/ecore_evas_wayland_common.c

index a6d5893..f9d0075 100644 (file)
@@ -33,7 +33,7 @@ static Ecore_Event_Handler *_ecore_evas_wl_event_hdls[13];
 static void _ecore_evas_wayland_resize(Ecore_Evas *ee, int location);
 
 /* local functions */
-static void 
+static void
 _ecore_evas_wl_common_state_update(Ecore_Evas *ee)
 {
    if (ee->func.fn_state_change) ee->func.fn_state_change(ee);
@@ -1994,7 +1994,7 @@ _ecore_evas_wl_common_new_internal(const char *disp_name, unsigned int parent, i
 
    if (!(method = evas_render_method_lookup(engine_name)))
      {
-        ERR("Render method lookup failed for Wayland_Shm");
+        ERR("Render method lookup failed for %s", engine_name);
         return NULL;
      }