[sr-dev] [tracker] Task opened: modules/lcr just needs some "\n" on the LM_DBG statements

[sip-router]

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below. 

User who did this - Kevin Scott Adams (TheGrandWazoo) 

Attached to Project - sip-router
Summary - modules/lcr just needs some "\n" on the LM_DBG statements
Task Type - Improvement
Category - Module
Status - Unconfirmed
Assigned To - 
Operating System - Linux
Severity - Low
Priority - Normal
Reported Version - 4.0
Due in Version - Undecided
Due Date - Undecided
Details - Diff file for modules/lcr for debug statements that are missing "\n".



--- kamailio-4.0.0/modules/lcr/lcr_mod.c        2013-02-27 03:23:41.000000000 -0500
+++ kamailio-4.0.3_sca/modules/lcr/lcr_mod.c    2013-08-21 08:49:38.303002393 -0400
@@ -1235,7 +1235,7 @@
        request_uri_re = from_uri_re = 0;

        do {
-           LM_DBG("loading, cycle %d with <%d> rows", n++, RES_ROW_N(res));
+           LM_DBG("loading, cycle %d with <%d> rows\n", n++, RES_ROW_N(res));
            for (i = 0; i < RES_ROW_N(res); i++) {

                request_uri_re = from_uri_re = 0;
@@ -1447,7 +1447,7 @@

        n = 0;
        do {
-           LM_DBG("loading, cycle %d with <%d> rows", n++, RES_ROW_N(res));
+           LM_DBG("loading, cycle %d with <%d> rows\n", n++, RES_ROW_N(res));
            for (i = 0; i < RES_ROW_N(res); i++) {
                row = RES_ROWS(res) + i;
                if ((VAL_NULL(ROW_VALUES(row)) == 1) ||
@@ -1898,7 +1898,7 @@
            if ((rule->from_uri_len != 0) &&
                (pcre_exec(rule->from_uri_re, NULL, from_uri.s,
                           from_uri.len, 0, 0, NULL, 0) < 0)) {
-               LM_DBG("from uri <%.*s> did not match to from regex <%.*s>",
+               LM_DBG("from uri <%.*s> did not match to from regex <%.*s>\n",
                       from_uri.len, from_uri.s, rule->from_uri_len,
                       rule->from_uri);
                goto next;
@@ -1908,7 +1908,7 @@
            if ((rule->request_uri_len != 0) &&
                (pcre_exec(rule->request_uri_re, NULL, request_uri->s,
                           request_uri->len, 0, 0, NULL, 0) < 0)) {
-               LM_DBG("request uri <%.*s> did not match to request regex <%.*s>",
+               LM_DBG("request uri <%.*s> did not match to request regex <%.*s>\n",
                       request_uri->len, request_uri->s, rule->request_uri_len,
                       rule->request_uri);
                goto next;
@@ -2284,7 +2284,7 @@
        delete_avp(defunct_gw_avp_type, defunct_gw_avp);
        val.n = gw_index;
        add_avp(defunct_gw_avp_type, defunct_gw_avp, val);
-       LM_DBG("added defunct_gw_avp <%u>", addr.u.addr32[0]);
+       LM_DBG("added defunct_gw_avp <%u>\n", addr.u.addr32[0]);
     }

     return 1;


More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=334

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.