davemds pushed a commit to branch master.

http://git.enlightenment.org/bindings/python/python-efl.git/commit/?id=f872e7b8c6baa20b485d5e6f262d326c0acb1565

commit f872e7b8c6baa20b485d5e6f262d326c0acb1565
Author: Dave Andreoli <d...@gurumeditation.it>
Date:   Fri Oct 14 22:45:32 2016 +0200

    Fix usage of EINA_LOG_* functions
    
    spotted by warnings :)
---
 efl/elementary/__init__.pyx        | 12 +++------
 efl/elementary/object_item.pxi     |  3 +--
 efl/eo/efl.eo.pyx                  |  4 +--
 efl/evas/efl.evas.pyx              |  4 +--
 efl/evas/efl.evas_object_smart.pxi | 50 +++++++++++++++++++-------------------
 efl/utils/logger.pyx               |  2 +-
 6 files changed, 35 insertions(+), 40 deletions(-)

diff --git a/efl/elementary/__init__.pyx b/efl/elementary/__init__.pyx
index d2b84c5..91081d2 100644
--- a/efl/elementary/__init__.pyx
+++ b/efl/elementary/__init__.pyx
@@ -713,8 +713,7 @@ def init():
         not carry any penalty though.
 
     """
-    EINA_LOG_DOM_INFO(PY_EFL_ELM_LOG_DOMAIN,
-        "Initializing efl.elementary", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_ELM_LOG_DOMAIN, "Initializing efl.elementary")
 
     # argc and argv are currently used by EFL to support app restart
     # and binary relocation. These are probably not useful to us but we
@@ -761,8 +760,7 @@ def shutdown():
         not carry any penalty though.
 
     """
-    EINA_LOG_DOM_INFO(PY_EFL_ELM_LOG_DOMAIN,
-        "Shutting down efl.elementary", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_ELM_LOG_DOMAIN, "Shutting down efl.elementary")
     return elm_shutdown()
 
 
@@ -880,8 +878,7 @@ def run():
     running the main (event/processing) loop for Elementary.
 
     """
-    EINA_LOG_DOM_DBG(PY_EFL_ELM_LOG_DOMAIN,
-        "Starting up main loop.", NULL)
+    EINA_LOG_DOM_DBG(PY_EFL_ELM_LOG_DOMAIN, "Starting up main loop.")
     with nogil:
         elm_run()
 
@@ -901,8 +898,7 @@ def exit():
         for you.
 
     """
-    EINA_LOG_DOM_DBG(PY_EFL_ELM_LOG_DOMAIN,
-        "Ending main loop.", NULL)
+    EINA_LOG_DOM_DBG(PY_EFL_ELM_LOG_DOMAIN, "Ending main loop.")
     elm_exit()
 
 def policy_set(Elm_Policy policy, value):
diff --git a/efl/elementary/object_item.pxi b/efl/elementary/object_item.pxi
index ac278ad..1f296bd 100644
--- a/efl/elementary/object_item.pxi
+++ b/efl/elementary/object_item.pxi
@@ -52,8 +52,7 @@ cdef _object_item_to_python(Elm_Object_Item *it):
 
     if data == NULL:
         # Create a dummy object item.
-        EINA_LOG_DOM_WARN(PY_EFL_ELM_LOG_DOMAIN,
-            "Creating an incomplete ObjectItem.", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_ELM_LOG_DOMAIN, "Creating an incomplete 
ObjectItem.")
         item = ObjectItem.__new__(ObjectItem)
         item._set_obj(it)
     else:
diff --git a/efl/eo/efl.eo.pyx b/efl/eo/efl.eo.pyx
index faa1478..195ee05 100644
--- a/efl/eo/efl.eo.pyx
+++ b/efl/eo/efl.eo.pyx
@@ -60,11 +60,11 @@ import atexit
 ######################################################################
 
 def init():
-    EINA_LOG_DOM_INFO(PY_EFL_EO_LOG_DOMAIN, "Initializing efl.eo", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_EO_LOG_DOMAIN, "Initializing efl.eo")
     return efl_object_init()
 
 def shutdown():
-    EINA_LOG_DOM_INFO(PY_EFL_EO_LOG_DOMAIN, "Shutting down efl.eo", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_EO_LOG_DOMAIN, "Shutting down efl.eo")
     return efl_object_shutdown()
 
 init()
diff --git a/efl/evas/efl.evas.pyx b/efl/evas/efl.evas.pyx
index 8bc7e57..b84f513 100644
--- a/efl/evas/efl.evas.pyx
+++ b/efl/evas/efl.evas.pyx
@@ -1213,7 +1213,7 @@ FILL_VERT  = 0.5, EVAS_HINT_FILL
 
 
 def init():
-    EINA_LOG_DOM_INFO(PY_EFL_EVAS_LOG_DOMAIN, "Initializing efl.evas", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_EVAS_LOG_DOMAIN, "Initializing efl.evas")
 
     if evas_object_event_callbacks_len != enums.EVAS_CALLBACK_LAST:
         raise SystemError("Number of object callbacks changed from %d to %d." %
@@ -1225,7 +1225,7 @@ def init():
 
 
 def shutdown():
-    EINA_LOG_DOM_INFO(PY_EFL_EVAS_LOG_DOMAIN, "Shutting down efl.evas", NULL)
+    EINA_LOG_DOM_INFO(PY_EFL_EVAS_LOG_DOMAIN, "Shutting down efl.evas")
     return evas_shutdown()
 
 
diff --git a/efl/evas/efl.evas_object_smart.pxi 
b/efl/evas/efl.evas_object_smart.pxi
index dc3c396..cd03495 100644
--- a/efl/evas/efl.evas_object_smart.pxi
+++ b/efl/evas/efl.evas_object_smart.pxi
@@ -39,7 +39,7 @@ cdef list _descriptions_to_list(const 
Evas_Smart_Cb_Description **arr, unsigned
         ret.append(SmartCbDescription.create(arr[i]))
 
     if arr[i+1] != NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "array was not NULL 
terminated!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "array was not NULL 
terminated!")
 
     return ret
 
@@ -102,13 +102,13 @@ cdef void _smart_object_delete(Evas_Object *o) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -127,13 +127,13 @@ cdef void _smart_object_move(Evas_Object *o, Evas_Coord 
x, Evas_Coord y) with gi
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -152,13 +152,13 @@ cdef void _smart_object_resize(Evas_Object *o, Evas_Coord 
w, Evas_Coord h) with
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -177,13 +177,13 @@ cdef void _smart_object_show(Evas_Object *o) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -202,13 +202,13 @@ cdef void _smart_object_hide(Evas_Object *o) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -227,13 +227,13 @@ cdef void _smart_object_color_set(Evas_Object *o, int r, 
int g, int b, int a) wi
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -253,13 +253,13 @@ cdef void _smart_object_clip_set(Evas_Object *o, 
Evas_Object *clip) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -280,13 +280,13 @@ cdef void _smart_object_clip_unset(Evas_Object *o) with 
gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -305,13 +305,13 @@ cdef void _smart_object_calculate(Evas_Object *o) with 
gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -331,13 +331,13 @@ cdef void _smart_object_member_add(Evas_Object *o, 
Evas_Object *clip) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <SmartObject>tmp
@@ -359,13 +359,13 @@ cdef void _smart_object_member_del(Evas_Object *o, 
Evas_Object *clip) with gil:
 
     tmp = evas_smart_data_get(evas_object_smart_smart_get(o))
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "cls is NULL!")
         return
     cls = <Smart>tmp
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_WARN(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         obj = None
     else:
         obj = <Eo>tmp
@@ -395,7 +395,7 @@ cdef object _smart_cb_pass_conv(void *addr):
 
 cdef void _smart_callback(void *data, Evas_Object *o, void *event_info) with 
gil:
     if data == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "data is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "data is NULL!")
         return
 
     cdef:
@@ -408,7 +408,7 @@ cdef void _smart_callback(void *data, Evas_Object *o, void 
*event_info) with gil
 
     tmp = efl_key_data_get(o, "python-eo")
     if tmp == NULL:
-        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!", NULL)
+        EINA_LOG_DOM_ERR(PY_EFL_EVAS_LOG_DOMAIN, "obj is NULL!")
         return
     else:
         obj = <SmartObject>tmp
diff --git a/efl/utils/logger.pyx b/efl/utils/logger.pyx
index 34d4ba7..1509132 100644
--- a/efl/utils/logger.pyx
+++ b/efl/utils/logger.pyx
@@ -125,4 +125,4 @@ rootlog.propagate = False
 cdef public int PY_EFL_LOG_DOMAIN = rootlog.eina_log_domain
 
 def logger_test_dbg():
-    EINA_LOG_DOM_DBG(PY_EFL_LOG_DOMAIN, "test message", NULL)
+    EINA_LOG_DOM_DBG(PY_EFL_LOG_DOMAIN, "test message")

-- 


Reply via email to