ecore_cocoa: create a log domain and use Eina_Log.
authorNicolas Aguirre <aguirre.nicolas@gmail.com>
Thu, 29 Jan 2015 16:08:23 +0000 (17:08 +0100)
committerCedric BAIL <cedric@osg.samsung.com>
Thu, 5 Feb 2015 15:38:07 +0000 (16:38 +0100)
Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/lib/ecore_cocoa/ecore_cocoa.m
src/lib/ecore_cocoa/ecore_cocoa_private.h
src/lib/ecore_cocoa/ecore_cocoa_window.m

index 1a094bcbc72a0ab1c9e05351ed819b1735cf8bbd..cfae44d982abfdff3f67658f0a943063b6e7a10e 100644 (file)
@@ -25,6 +25,8 @@ static int _ecore_cocoa_init_count = 0;
 
 static int old_flags;
 
+static int _ecore_cocoa_log_domain = -1;
+
 EAPI int
 ecore_cocoa_init(void)
 {
@@ -37,6 +39,13 @@ ecore_cocoa_init(void)
    if (!ecore_event_init())
      return --_ecore_cocoa_init_count;
 
+   _ecore_cocoa_log_domain = eina_log_domain_register("ecore_cocoa", ECORE_DEFAULT_LOG_COLOR);
+   if(_ecore_cocoa_log_domain < 0)
+     {
+        EINA_LOG_ERR("Unable to create a log domain for ecore_cocoa.");
+        return 0;
+     }
+
    ECORE_COCOA_EVENT_GOT_FOCUS  = ecore_event_type_new();
    ECORE_COCOA_EVENT_LOST_FOCUS = ecore_event_type_new();
    ECORE_COCOA_EVENT_RESIZE     = ecore_event_type_new();
@@ -64,6 +73,7 @@ ecore_cocoa_shutdown(void)
    if (--_ecore_cocoa_init_count != 0)
      return _ecore_cocoa_init_count;
 
+   eina_log_domain_unregister(_ecore_cocoa_log_domain);
    ecore_event_shutdown();
 
    return _ecore_cocoa_init_count;
@@ -80,7 +90,7 @@ _ecore_cocoa_event_modifiers(unsigned int mod)
    if(mod & NSCommandKeyMask) modifiers |= ECORE_EVENT_MODIFIER_WIN;
    if(mod & NSNumericPadKeyMask) modifiers |= ECORE_EVENT_LOCK_NUM;
 
-   printf("key modifiers: %d, %d\n", mod, modifiers);
+   DBG("key modifiers: %d, %d\n", mod, modifiers);
    return modifiers;
 }
 
@@ -286,7 +296,7 @@ ecore_cocoa_feed_events(void *anEvent)
            {
               if (keystable[i].code == [keychar characterAtIndex:0])
               {
-                 printf("Key pressed : %s\n", keystable[i].name);
+                 DBG("Key pressed : %s\n", keystable[i].name);
                  ev->keyname = keystable[i].name;
                  ev->key = keystable[i].name;
                  ev->string = keystable[i].compose;
@@ -311,7 +321,7 @@ ecore_cocoa_feed_events(void *anEvent)
          EcoreCocoaWindow *window = (EcoreCocoaWindow *)[event window];
          NSString *keychar = [event characters];
 
-         printf("Key Up\n");
+         DBG("Key Up\n");
 
          ev = calloc(1, sizeof (Ecore_Event_Key));
          if (!ev) return pass;
@@ -432,7 +442,7 @@ ecore_cocoa_feed_events(void *anEvent)
       }
       case NSScrollWheel:
       {
-         printf("Scroll Wheel\n");
+         DBG("Scroll Wheel\n");
          break;
       }
       default:
index 41310d43d5fe4b26c00784f4d90f4f840fc151e7..9127de15c6aad650219cc0b0f71c27cabe97b0d9 100644 (file)
@@ -1,6 +1,36 @@
 #ifndef _ECORE_COCOA_PRIVATE_H
 #define _ECORE_COCOA_PRIVATE_H
 
+#include "ecore_cocoa_window.h"
+
+extern int _ecore_cocoa_log_domain;
+
+#ifdef ERR
+# undef ERR
+#endif
+#define ERR(...) EINA_LOG_DOM_ERR(_ecore_cocoa_log_domain, __VA_ARGS__)
+
+#ifdef DBG
+# undef DBG
+#endif
+#define DBG(...) EINA_LOG_DOM_DBG(_ecore_cocoa_log_domain, __VA_ARGS__)
+
+#ifdef INF
+# undef INF
+#endif
+#define INF(...) EINA_LOG_DOM_INFO(_ecore_cocoa_log_domain, __VA_ARGS__)
+
+#ifdef WRN
+# undef WRN
+#endif
+#define WRN(...) EINA_LOG_DOM_WARN(_ecore_cocoa_log_domain, __VA_ARGS__)
+
+#ifdef CRI
+# undef CRI
+#endif
+#define CRI(...) EINA_LOG_DOM_CRIT(_ecore_cocoa_log_domain, __VA_ARGS__)
+
+
 struct _Ecore_Cocoa_Window
 {
    EcoreCocoaWindow *window;
index 8300fbba3ae08396107c0b3342bfd87fdca3317c..5e8a2052fdede863645acb484d79cd3908d3dd78 100644 (file)
@@ -4,6 +4,7 @@
 \r
 #include <Ecore.h>\r
 #include <Ecore_Cocoa.h>\r
+#include "ecore_cocoa_private.h"\r
 #import "ecore_cocoa_window.h"\r
 \r
 @implementation EcoreCocoaWindow\r
@@ -58,8 +59,7 @@
    event = malloc(sizeof(Ecore_Cocoa_Event_Video_Resize));\r
    if (event == NULL)\r
      {\r
-        // FIXME Use Eina_Log\r
-        printf("Failed to allocate Ecore_Cocoa_Event_Video_Resize\n");\r
+        DBG("Failed to allocate Ecore_Cocoa_Event_Video_Resize\n");\r
         return;\r
      }\r
    event->w = size.width;\r
@@ -77,7 +77,7 @@
   e = malloc(sizeof(Ecore_Cocoa_Event_Window));\r
   if (!e)\r
     {\r
-      printf("GOT_FOCUS: Failed to allocate Ecore_Cocoa_Event_Window\n");\r
+      DBG("GOT_FOCUS: Failed to allocate Ecore_Cocoa_Event_Window\n");\r
       return;\r
     }\r
   e->wid = [notification object];\r
@@ -91,7 +91,7 @@
   e = malloc(sizeof(Ecore_Cocoa_Event_Window));\r
   if (!e)\r
     {\r
-      printf("LOST_FOCUS: Failed to allocate Ecore_Cocoa_Event_Window\n");\r
+      DBG("LOST_FOCUS: Failed to allocate Ecore_Cocoa_Event_Window\n");\r
       return;\r
     }\r
   e->wid = [notification object];\r
@@ -222,7 +222,7 @@ ecore_cocoa_window_show(Ecore_Cocoa_Window *window)
 {\r
   if (!window || [window->window isVisible])\r
     {\r
-      printf("Window(%p) is not visible\n", window->window);\r
+      DBG("Window(%p) is not visible\n", window->window);\r
       return;\r
     }\r
 \r
@@ -276,5 +276,8 @@ Ecore_Cocoa_Window_Id ecore_cocoa_window_get_window_id(Ecore_Cocoa_Window *windo
 {\r
   if (!window)\r
     return 0;\r
+\r
+  DBG("Return : %p", window->window);\r
+\r
   return window->window;\r
 }\r