diff options
| author | Marcel Hollerbach <mail@marcel-hollerbach.de> | 2020-03-30 16:49:05 +0200 |
|---|---|---|
| committer | Stefan Schmidt <s.schmidt@samsung.com> | 2020-03-31 14:56:43 +0200 |
| commit | 455b07ec62e42c246095d78d2506343761480a28 (patch) | |
| tree | ee500d3dc2890cba6849b3da49a92fe3fb6fa574 | |
| parent | 624ea389c932271fffbc260ada29a2dbadd51603 (diff) | |
| download | efl-455b07ec62e42c246095d78d2506343761480a28.tar.gz | |
exactness: equip with ERR and CRIT
before a few commits, we had the situation that errors were overseen
because the log was simply so big, that errors did not get shown
properly.
With this commit, exactness will simply abort if there is a real issue
in the code.
Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Differential Revision: https://phab.enlightenment.org/D11624
| -rw-r--r-- | src/bin/exactness/exactness.c | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/src/bin/exactness/exactness.c b/src/bin/exactness/exactness.c index 8338c12cd5..6599ef7741 100644 --- a/src/bin/exactness/exactness.c +++ b/src/bin/exactness/exactness.c @@ -20,6 +20,13 @@ #define BUF_SIZE 1024 +#define DBG(...) EINA_LOG_DOM_DBG(_log_domain, __VA_ARGS__) +#define INF(...) EINA_LOG_DOM_INFO(_log_domain, __VA_ARGS__) +#define ERR(...) EINA_LOG_DOM_ERR(_log_domain, __VA_ARGS__) +#define CRI(...) EINA_LOG_DOM_CRIT(_log_domain, __VA_ARGS__) + +static int _log_domain = -1; + typedef struct { EINA_INLIST; @@ -63,8 +70,7 @@ _image_load(const char *filename) err = evas_object_image_load_error_get(img); if (err != EVAS_LOAD_ERROR_NONE) { - fprintf(stderr, "could not load image '%s'. error string is \"%s\"\n", - filename, evas_load_error_str(err)); + CRI("Failed to load image"); return NULL; } @@ -174,7 +180,7 @@ _run_test_compare(const List_Entry *ent) { if (mkdir(origdir, 0744) < 0) { - fprintf(stderr, "Failed to create dir %s\n", origdir); + CRI("Failed to create dir %s\n", origdir); return; } } @@ -219,7 +225,7 @@ _run_command_prepare(const List_Entry *ent, char *buf) sprintf(scn_path, "%s/%s.exu", base_dir, ent->name); if (ecore_file_exists(scn_path)) goto ok; } - fprintf(stderr, "Test %s not found in the provided base directories\n", ent->name); + CRI("Test %s not found in the provided base directories\n", ent->name); return EINA_FALSE; ok: sbuf = eina_strbuf_new(); @@ -320,7 +326,7 @@ _job_consume() if (!ecore_exe_pipe_run(buf, ECORE_EXE_TERM_WITH_PARENT, ent)) { - fprintf(stderr, "Failed executing test '%s'\n", ent->name); + CRI("Failed executing test '%s'\n", ent->name); } } _next_test_to_run = EINA_INLIST_CONTAINER_GET( @@ -475,9 +481,13 @@ main(int argc, char *argv[]) if (!ecore_evas_init()) return EXIT_FAILURE; + _log_domain = eina_log_domain_register("exactness", "red"); _dest_dir = "./"; _scan_objs = scan_objs; + eina_log_abort_on_critical_set(EINA_TRUE); + eina_log_abort_on_critical_level_set(EINA_LOG_LEVEL_ERR); + args = ecore_getopt_parse(&optdesc, values, argc, argv); if (args < 0) { |