better log.

* lower case domain names;

 * binaries use their own color (EDJE_CC_DEFAULT_LOG_COLOR)

 * log messages do not take multiple lines, it's annoying. The colors
   should call your attention already.

 * the ever annoying "did you forgot fixed: 1 1;" message that used to
   consume 3 lines is now bit more descriptive and uses a single line.



SVN revision: 53178

legacy/edje/src/bin/edje_cc.c

@@ -61,8 +61,9 @@ main(int argc, char **argv)
    if (!eina_init())
      return -1;
 
-   _edje_cc_log_dom = eina_log_domain_register("EDJE_CC", EDJE_DEFAULT_LOG_COLOR);
+   _edje_cc_log_dom = eina_log_domain_register
-   if(_edje_cc_log_dom<0)
+     ("edje_cc", EDJE_CC_DEFAULT_LOG_COLOR);
+   if (_edje_cc_log_dom < 0)
      {
        EINA_LOG_ERR("Enable to create a log domain.");
        exit(-1);

legacy/edje/src/bin/edje_convert_main.c

@@ -118,8 +118,9 @@ main(int argc, char **argv)
    if (!eina_init())
      return -1;
 
-   _edje_cc_log_dom = eina_log_domain_register("EDJE_CONVERT", EDJE_DEFAULT_LOG_COLOR);
+   _edje_cc_log_dom = eina_log_domain_register
-   if(_edje_cc_log_dom<0)
+     ("edje_convert", EDJE_CC_DEFAULT_LOG_COLOR);
+   if (_edje_cc_log_dom < 0)
      {
        EINA_LOG_ERR("Enable to create a log domain.");
        exit(-1);

legacy/edje/src/bin/edje_decc.c

@@ -61,10 +61,11 @@ main(int argc, char **argv)
    setlocale(LC_NUMERIC, "C");
    if (!eina_init())
      exit(-1);
-   _edje_cc_log_dom = eina_log_domain_register("edje_decc", EDJE_CC_DEFAULT_LOG_COLOR);
+   _edje_cc_log_dom = eina_log_domain_register
-   if(_edje_cc_log_dom < 0)
+     ("edje_decc", EDJE_CC_DEFAULT_LOG_COLOR);
+   if (_edje_cc_log_dom < 0)
      {
-       EINA_LOG_ERR("Edje_decc: Impossible to create a log domain for edje_decc");
+       EINA_LOG_ERR("Impossible to create a log domain.");
        eina_shutdown();
        exit(-1);
      }

legacy/edje/src/lib/edje_calc.c

@@ -1675,8 +1675,8 @@ _edje_part_recalc(Edje *ed, Edje_Real_Part *ep, int flags)
 	  faxes = "X";
 	else if ((flags & FLAG_Y))
 	  faxes = "Y";
-	ERR("Circular dependency when calculating part \"%s\"\n"
+	ERR("Circular dependency when calculating part \"%s\". "
-	    "Already calculating %s [%02x] axes\n"
+	    "Already calculating %s [%02x] axes. "
 	    "Need to calculate %s [%02x] axes",
 	    ep->part->name,
 	    axes, ep->calculating,

legacy/edje/src/lib/edje_convert.c

@@ -37,19 +37,18 @@ _edje_collection_string_free(void *data)
 	edf = (Edje_File*) _edje_file_get();
 
 	if (!edf->warning)
-	  ERR("EDJE ERROR:\n"
+	  CRITICAL("This program as probably called edje_shutdown() with "
-	      "\n"
+                   "active Edje objects still around. "
-	      "Naughty Programmer - spank spank!\n"
+                   "This can cause problems as both Evas and Edje retain "
-	      "\n"
+                   "references to the objects. "
-	      "This program as probably called edje_shutdown() with active Edje objects\n"
+                   "You should shut down all canvases and objects "
-	      "still around.\n This can cause problems as both Evas and Edje retain\n"
+                   "before calling edje_shutdown(). "
-	      "references to the objects. you should shut down all canvases and objects\n"
+                   "The following errors are the edje object files and "
-	      "before calling edje_shutdown().\n"
+                   "parts that are still hanging around, with their reference "
-	      "The following errors are the edje object files and parts that are still\n"
+                   "counts");
-	      "hanging around, with their reference counts");
 
 	edf->warning = 1;
-	ERR("EEK: EDJE FILE: \"%s\" ref(%i) PART: \"%s\" ref(%i) ",
+	ERR("file: '%s', references: %i, part: '%s', references: %i",
 	    edf->path, edf->references,
 	    ce->ref->part, ce->ref->references);
 

legacy/edje/src/lib/edje_edit.c

@@ -4202,7 +4202,7 @@ edje_edit_font_add(Evas_Object *obj, const char* path, const char* alias)
 
    GET_ED_OR_RETURN(EINA_FALSE);
 
-   INF("ADD FONT: %s\n", path);
+   INF("ADD FONT: %s", path);
 
    if (!path) return EINA_FALSE;
    if (stat(path, &st) || !S_ISREG(st.st_mode)) return EINA_FALSE;
@@ -4268,7 +4268,7 @@ edje_edit_font_del(Evas_Object *obj, const char* alias)
 
    GET_ED_OR_RETURN(EINA_FALSE);
 
-   INF("DEL FONT: %s\n", alias);
+   INF("DEL FONT: %s", alias);
 
    if (!alias) return EINA_FALSE;
    if (!ed->file) return EINA_FALSE;
@@ -6887,14 +6887,12 @@ edje_edit_print_internal_status(Evas_Object *obj)
    _edje_generate_source(obj);
    return;
 
-   INF("\n****** CHECKIN' INTERNAL STRUCTS STATUS *********");
+   INF("****** CHECKIN' INTERNAL STRUCTS STATUS *********");
 
-   INF("*** Edje\n");
+   INF("path: '%s', group: '%s', parent: '%s'",
-   INF("    path: '%s'", ed->path);
+       ed->path, ed->group, ed->parent);
-   INF("    group: '%s'", ed->group);
-   INF("    parent: '%s'", ed->parent);
 
-   INF("*** Parts [table:%d list:%d]", ed->table_parts_size,
+   INF("Parts [table:%d list:%d]", ed->table_parts_size,
        ed->collection->parts_count);
    for (i = 0; i < ed->collection->parts_count; ++i)
      {
@@ -6904,14 +6902,14 @@ edje_edit_print_internal_status(Evas_Object *obj)
 	p = ed->collection->parts[i];
 
 	rp = ed->table_parts[p->id % ed->table_parts_size];
-	printf("    [%d]%s ", p->id, p->name);
+	printf("part[%d]: '%s' ", p->id, p->name);
 	if (p == rp->part)
-	  printf(" OK!\n");
+	  printf("OK!\n");
 	else
-	  WRN(" WRONG (table[%id]->name = '%s')", p->id, rp->part->name);
+	  printf("WRONG (table[%d]->name = '%s')\n", p->id, rp->part->name);
      }
 
-   INF("*** Programs [table:%d list:%d,%d,%d,%d,%d]", ed->table_programs_size,
+   INF("Programs [table:%d list:%d,%d,%d,%d,%d]", ed->table_programs_size,
        ed->collection->programs.fnmatch_count,
        ed->collection->programs.strcmp_count,
        ed->collection->programs.strncmp_count,
@@ -6920,10 +6918,8 @@ edje_edit_print_internal_status(Evas_Object *obj)
    for(j = 0; j < ed->table_programs_size; ++j)
      {
 	epr = ed->table_programs[i % ed->table_programs_size];
-	printf("     [%d]%s\n", epr->id, epr->name);
+	printf("program[%d]: '%s'\n", epr->id, epr->name);
      }
 
-   printf("\n");
+   INF("******************  END  ************************");
-
-   INF("******************  END  ************************\n");
 }

legacy/edje/src/lib/edje_embryo.c

@@ -3031,12 +3031,13 @@ _edje_embryo_test_run(Edje *ed, const char *fname, const char *sig, const char *
 	ret = embryo_program_run(ed->collection->script, fn);
 	if (ret == EMBRYO_PROGRAM_FAIL)
 	  {
- 	     ERR("ERROR with embryo script.\n"
+ 	     ERR("ERROR with embryo script. "
-                 "OBJECT NAME: %s\n"
+                 "OBJECT NAME: '%s', "
-                 "OBJECT FILE: %s\n"
+                 "OBJECT FILE: '%s', "
-                 "ENTRY POINT: %s\n"
+                 "ENTRY POINT: '%s', "
-                 "SIGNAL/SRC:  %s / %s\n"
+                 "SIGNAL: '%s', "
-		 "ERROR:       %s",
+                 "SOURCE: '%s', "
+		 "ERROR: '%s'",
                  ed->collection->part,
                  ed->file->path,
                  fname,
@@ -3045,12 +3046,13 @@ _edje_embryo_test_run(Edje *ed, const char *fname, const char *sig, const char *
 	  }
 	else if (ret == EMBRYO_PROGRAM_TOOLONG)
 	  {
-	     ERR("ERROR with embryo script.\n"
+	     ERR("ERROR with embryo script. "
-                 "OBJECT NAME: %s\n"
+                 "OBJECT NAME: '%s', "
-                 "OBJECT FILE: %s\n"
+                 "OBJECT FILE: '%s', "
-                 "ENTRY POINT: %s\n"
+                 "ENTRY POINT: '%s', "
-                 "SIGNAL/SRC:  %s / %s\n"
+                 "SIGNAL: '%s', "
-		 "ERROR:       Script exceeded maximum allowed cycle count of %i",
+                 "SOURCE: '%s', "
+		 "ERROR: 'Script exceeded maximum allowed cycle count of %i'",
                  ed->collection->part,
                  ed->file->path,
 		 fname,

legacy/edje/src/lib/edje_load.c

@@ -461,7 +461,7 @@ _edje_object_file_set_internal(Evas_Object *obj, const char *file, const char *g
 			rp->object = evas_object_table_add(ed->evas);
 			break;
 		     case EDJE_PART_TYPE_GRADIENT:
-			ERR("SPANK ! SPANK ! SPANK !\nYOU ARE USING GRADIENT IN PART %s FROM GROUP %s INSIDE FILE %s !!\n THEY ARE NOW REMOVED !",
+			ERR("SPANK ! SPANK ! SPANK ! YOU ARE USING GRADIENT IN PART %s FROM GROUP %s INSIDE FILE %s !! THEY ARE NOW REMOVED !",
 			    ep->name, group, file);
 		     default:
 			ERR("wrong part type %i!", ep->type);

legacy/edje/src/lib/edje_lua.c

@@ -120,7 +120,7 @@ jmp_buf _edje_lua_panic_jmp;
 static int
 _edje_lua_custom_panic(__UNUSED__ lua_State *L)
 {
-   printf("PANIC\n");
+   CRITICAL("PANIC");
    longjmp(_edje_lua_panic_jmp, 1);
    return 1; /* longjmp() never returns, but this keep gcc happy */
 }
@@ -5049,7 +5049,7 @@ _edje_lua_alloc(void *ud, void *ptr, size_t osize, size_t nsize)
    /* ANSI requires that realloc(NULL, size) == malloc(size) */
    ptr2 = realloc(ptr, nsize);
    if (ptr2) return ptr2;
-   ERR("Edje Lua cannot re-allocate %zu bytes\n", nsize);
+   ERR("Edje Lua cannot re-allocate %zu bytes", nsize);
    return ptr2;
 }
 

legacy/edje/src/lib/edje_main.c

@@ -55,7 +55,8 @@ edje_init(void)
 	return --_edje_init_count;
      }
 
-   _edje_default_log_dom = eina_log_domain_register("Edje", EDJE_DEFAULT_LOG_COLOR);
+   _edje_default_log_dom = eina_log_domain_register
+     ("edje", EDJE_DEFAULT_LOG_COLOR);
    if (_edje_default_log_dom < 0)
      {
 	EINA_LOG_ERR("Edje Can not create a general log domain.");

legacy/edje/src/lib/edje_message_queue.c

@@ -687,11 +687,11 @@ _edje_message_process(Edje_Message *em)
    ret = embryo_program_run(em->edje->collection->script, fn);
    if (ret == EMBRYO_PROGRAM_FAIL)
      {
-        ERR("ERROR with embryo script.\n"
+        ERR("ERROR with embryo script. "
-            "OBJECT NAME: %s\n"
+            "OBJECT NAME: '%s', "
-            "OBJECT FILE: %s\n"
+            "OBJECT FILE: '%s', "
-            "ENTRY POINT: %s\n"
+            "ENTRY POINT: '%s', "
-            "ERROR:       %s",
+            "ERROR: '%s'",
             em->edje->collection->part,
             em->edje->file->path,
             "message",
@@ -699,11 +699,11 @@ _edje_message_process(Edje_Message *em)
      }
    else if (ret == EMBRYO_PROGRAM_TOOLONG)
      {
-        ERR("ERROR with embryo script.\n"
+        ERR("ERROR with embryo script. "
-            "OBJECT NAME: %s\n"
+            "OBJECT NAME: '%s', "
-            "OBJECT FILE: %s\n"
+            "OBJECT FILE: '%s', "
-            "ENTRY POINT: %s\n"
+            "ENTRY POINT: '%s', "
-            "ERROR:       Script exceeded maximum allowed cycle count of %i",
+            "ERROR: 'Script exceeded maximum allowed cycle count of %i'",
             em->edje->collection->part,
             em->edje->file->path,
             "message",

legacy/edje/src/lib/edje_script_only.c

@@ -618,11 +618,11 @@ _call_fn(Edje * ed, const char *fname, Embryo_Function fn)
    ret = embryo_program_run(ed->collection->script, fn);
    if (ret == EMBRYO_PROGRAM_FAIL)
      {
-	ERR("ERROR with embryo script.\n"
+	ERR("ERROR with embryo script. "
-            "OBJECT NAME: %s\n"
+            "OBJECT NAME: '%s', "
-            "OBJECT FILE: %s\n"
+            "OBJECT FILE: '%s', "
-            "ENTRY POINT: %s\n"
+            "ENTRY POINT: '%s', "
-	    "ERROR:       %s",
+	    "ERROR: '%s'",
             ed->collection->part,
             ed->file->path,
             fname,
@@ -630,11 +630,11 @@ _call_fn(Edje * ed, const char *fname, Embryo_Function fn)
      }
    else if (ret == EMBRYO_PROGRAM_TOOLONG)
      {
-	ERR("ERROR with embryo script.\n"
+	ERR("ERROR with embryo script. "
-            "OBJECT NAME: %s\n"
+            "OBJECT NAME: '%s', "
-            "OBJECT FILE: %s\n"
+            "OBJECT FILE: '%s', "
-            "ENTRY POINT: %s\n"
+            "ENTRY POINT: '%s', "
-	    "ERROR:       Script exceeded maximum allowed cycle count of %i",
+	    "ERROR: 'Script exceeded maximum allowed cycle count of %i'",
             ed->collection->part,
             ed->file->path,
             fname, 

legacy/edje/src/lib/edje_util.c

@@ -2929,11 +2929,13 @@ edje_object_size_min_restricted_calc(Evas_Object *obj, Evas_Coord *minw, Evas_Co
 	  }
 	if ((ed->w > 4000) || (ed->h > 4000))
 	  {
-	     ERR("file %s, group %s has a non-fixed part. add fixed: 1 1; ???",
+             if (pep)
-		    ed->path, ed->group);
+               ERR("file %s, group %s has a non-fixed part '%s'. Adding 'fixed: 1 1;' to source EDC may help. Continuing discarding faulty part.",
-	     if (pep)
+                   ed->path, ed->group, pep->part->name);
-	       ERR("  Problem part is: %s", pep->part->name);
+             else
-	     ERR("  Will recalc min size not allowing broken parts to affect the result.");
+               ERR("file %s, group %s overflowed 4000x4000 with minimum size of %dx%d. Continuing discarding faulty parts.",
+                   ed->path, ed->group, ed->w, ed->h);
+
 	     if (reset_maxwh)
 	       {
 		  reset_maxwh = 0;

legacy/edje/src/lib/edje_var.c

@@ -33,20 +33,20 @@ _edje_var_timer_cb(void *data)
         ret = embryo_program_run(ed->collection->script, fn);
         if (ret == EMBRYO_PROGRAM_FAIL)
           {
-             ERR("ERROR with embryo script (timer callback).\n"
+             ERR("ERROR with embryo script (timer callback). "
-                 "OBJECT NAME: %s\n"
+                 "OBJECT NAME: '%s', "
-                 "OBJECT FILE: %s\n"
+                 "OBJECT FILE: '%s', "
-                 "ERROR:       %s",
+                 "ERROR: '%s'",
                  ed->collection->part,
                  ed->file->path,
                  embryo_error_string_get(embryo_program_error_get(ed->collection->script)));
           }
         else if (ret == EMBRYO_PROGRAM_TOOLONG)
           {
-             ERR("ERROR with embryo script (timer callback).\n"
+             ERR("ERROR with embryo script (timer callback). "
-                 "OBJECT NAME: %s\n"
+                 "OBJECT NAME: '%s', "
-                 "OBJECT FILE: %s\n"
+                 "OBJECT FILE: '%s', "
-                 "ERROR:       Script exceeded maximum allowed cycle count of %i",
+                 "ERROR: 'Script exceeded maximum allowed cycle count of %i'",
                  ed->collection->part,
                  ed->file->path,
                  embryo_program_max_cycle_run_get(ed->collection->script));
@@ -114,20 +114,20 @@ _edje_var_anim_cb(void *data __UNUSED__)
                             ret = embryo_program_run(ed->collection->script, fn);
                             if (ret == EMBRYO_PROGRAM_FAIL)
                               {
-                                 ERR("ERROR with embryo script (anim callback).\n"
+                                 ERR("ERROR with embryo script (anim callback). "
-                                     "OBJECT NAME: %s\n"
+                                     "OBJECT NAME: '%s', "
-                                     "OBJECT FILE: %s\n"
+                                     "OBJECT FILE: '%s', "
-                                     "ERROR:       %s",
+                                     "ERROR: '%s'",
                                      ed->collection->part,
                                      ed->file->path,
                                      embryo_error_string_get(embryo_program_error_get(ed->collection->script)));
                               }
                             else if (ret == EMBRYO_PROGRAM_TOOLONG)
                               {
-                                 ERR("ERROR with embryo script (anim callback).\n"
+                                 ERR("ERROR with embryo script (anim callback). "
-                                     "OBJECT NAME: %s\n"
+                                     "OBJECT NAME: '%s', "
-                                     "OBJECT FILE: %s\n"
+                                     "OBJECT FILE: '%s', "
-                                     "ERROR:       Script exceeded maximum allowed cycle count of %i",
+                                     "ERROR: 'Script exceeded maximum allowed cycle count of %i'",
                                      ed->collection->part,
                                      ed->file->path,
                                      embryo_program_max_cycle_run_get(ed->collection->script));