X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/c5bf8d0723c36491a1bf37a9f27858b43c4c6e2c..ce87516da629c9b6a350d8c28f806926863aa94a:/src/xbt/xbt_log_layout_format.cpp?ds=sidebyside diff --git a/src/xbt/xbt_log_layout_format.cpp b/src/xbt/xbt_log_layout_format.cpp index 854968ec1b..00e58fd526 100644 --- a/src/xbt/xbt_log_layout_format.cpp +++ b/src/xbt/xbt_log_layout_format.cpp @@ -1,12 +1,12 @@ /* layout_simple - a dumb log layout */ -/* Copyright (c) 2007-2019. The SimGrid Team. */ +/* Copyright (c) 2007-2020. The SimGrid Team. */ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ +#include "simgrid/engine.h" /* simgrid_get_clock */ #include "simgrid/host.h" -#include "simgrid/msg.h" /* MSG_get_clock */ #include "src/xbt/log_private.hpp" #include "xbt/sysdep.h" #include @@ -30,51 +30,55 @@ static constexpr const char* ERRMSG = } else \ return 0 -#define set_sz_from_precision() \ - if (1) { \ - sz = rem_size; \ - if (precision != -1) { \ - if (precision < sz) \ - sz = precision + 1; /* +1 for the final '\0' */ \ - precision = -1; \ - } \ - } else (void)0 +#define set_sz_from_precision() \ + if (true) { \ + sz = rem_size; \ + if (precision != -1) { \ + if (precision < sz) \ + sz = precision + 1; /* +1 for the final '\0' */ \ + precision = -1; \ + } \ + } else \ + (void)0 -#define show_it(data, letter) \ - if (1) { \ - int len; \ - int wd; \ - if (length == -1) { \ - wd = 0; \ - } else { \ - wd = length; \ - length = -1; \ - } \ - if (precision == -1) { \ - len = snprintf(p, rem_size, "%*" letter, wd, data); \ - } else { \ - len = snprintf(p, rem_size, "%*.*" letter, wd, precision, data); \ - precision = -1; \ - } \ - check_overflow(len); \ - } else (void)0 +#define show_it(data, letter) \ + if (true) { \ + int len; \ + int wd; \ + if (length == -1) { \ + wd = 0; \ + } else { \ + wd = length; \ + length = -1; \ + } \ + if (precision == -1) { \ + len = snprintf(p, rem_size, "%*" letter, wd, (data)); \ + } else { \ + len = snprintf(p, rem_size, "%*.*" letter, wd, precision, (data)); \ + precision = -1; \ + } \ + check_overflow(len); \ + } else \ + (void)0 -#define show_string(data) \ - if (1) { \ - const char *show_string_data = (data); \ - show_it(show_string_data ? show_string_data : "(null)", "s"); \ - } else (void)0 -#define show_int(data) show_it(data, "d") -#define show_double(data) show_it(data, "f") +#define show_string(data) \ + if (true) { \ + const char* show_string_data = (data); \ + show_it(show_string_data ? show_string_data : "(null)", "s"); \ + } else \ + (void)0 +#define show_int(data) show_it((data), "d") +#define show_double(data) show_it((data), "f") -static int xbt_log_layout_format_doit(xbt_log_layout_t l, xbt_log_event_t ev, const char* msg_fmt) +static int xbt_log_layout_format_doit(const s_xbt_log_layout_t* l, xbt_log_event_t ev, const char* msg_fmt) { char *p = ev->buffer; int rem_size = ev->buffer_size; int precision = -1; int length = -1; - for (char* q = static_cast(l->data) ; *q != '\0' ; q++) { + char* q = static_cast(l->data); + while (*q != '\0') { if (*q == '%') { q++; do { @@ -95,7 +99,7 @@ static int xbt_log_layout_format_doit(xbt_log_layout_t l, xbt_log_event_t ev, co check_overflow(1); break; case '.': /* precision specifier */ - precision = strtol(q + 1, &q, 10); + precision = static_cast(strtol(q + 1, &q, 10)); continue; /* conversion specifier still not found, continue reading */ case '0': case '1': @@ -107,7 +111,7 @@ static int xbt_log_layout_format_doit(xbt_log_layout_t l, xbt_log_event_t ev, co case '7': case '8': case '9': /* length modifier */ - length = strtol(q, &q, 10); + length = static_cast(strtol(q, &q, 10)); continue; /* conversion specifier still not found, continue reading */ case 'c': /* category name; LOG4J compliant should accept a precision postfix to show the hierarchy */ @@ -143,10 +147,10 @@ static int xbt_log_layout_format_doit(xbt_log_layout_t l, xbt_log_event_t ev, co show_string(ev->functionName); break; case 'd': /* date; LOG4J compliant */ - show_double(MSG_get_clock()); + show_double(simgrid_get_clock()); break; case 'r': /* application age; LOG4J compliant */ - show_double(MSG_get_clock()); + show_double(simgrid_get_clock()); break; case 'm': { /* user-provided message; LOG4J compliant */ int sz; @@ -163,28 +167,29 @@ static int xbt_log_layout_format_doit(xbt_log_layout_t l, xbt_log_event_t ev, co xbt_abort(); } break; /* done, continue normally */ - } while (1); + } while (true); } else { *p = *q; check_overflow(1); } + q++; } *p = '\0'; return 1; } -static void xbt_log_layout_format_free(xbt_log_layout_t lay) +static void xbt_log_layout_format_free(const s_xbt_log_layout_t* lay) { xbt_free(lay->data); } -xbt_log_layout_t xbt_log_layout_format_new(char *arg) +xbt_log_layout_t xbt_log_layout_format_new(const char* arg) { xbt_log_layout_t res = xbt_new0(s_xbt_log_layout_t, 1); res->do_layout = &xbt_log_layout_format_doit; res->free_ = &xbt_log_layout_format_free; - res->data = xbt_strdup((char *) arg); + res->data = xbt_strdup(arg); return res; }