Author: rjung
Date: Thu Jul 10 20:34:06 2014
New Revision: 1609566

URL: http://svn.apache.org/r1609566
Log:
Standardize on %d instead of %i on output.
Both should be equivalent for any type of printf,
like the one used in jk_log().

Modified:
    tomcat/jk/trunk/native/common/jk_ajp_common.c
    tomcat/jk/trunk/native/common/jk_lb_worker.c
    tomcat/jk/trunk/native/common/jk_status.c

Modified: tomcat/jk/trunk/native/common/jk_ajp_common.c
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/native/common/jk_ajp_common.c?rev=1609566&r1=1609565&r2=1609566&view=diff
==============================================================================
--- tomcat/jk/trunk/native/common/jk_ajp_common.c (original)
+++ tomcat/jk/trunk/native/common/jk_ajp_common.c Thu Jul 10 20:34:06 2014
@@ -1976,8 +1976,8 @@ static int ajp_process_callback(jk_msg_b
              */
             if (len > (unsigned int)(msg->len - 3)) {
                 jk_log(l, JK_LOG_ERROR,
-                       "Chunk length too large. Length of AJP message is %i,"
-                       " chunk length is %i.", msg->len, len);
+                       "Chunk length too large. Length of AJP message is %d,"
+                       " chunk length is %d.", msg->len, len);
                 JK_TRACE_EXIT(l);
                 return JK_INTERNAL_ERROR;
             }

Modified: tomcat/jk/trunk/native/common/jk_lb_worker.c
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/native/common/jk_lb_worker.c?rev=1609566&r1=1609565&r2=1609566&view=diff
==============================================================================
--- tomcat/jk/trunk/native/common/jk_lb_worker.c (original)
+++ tomcat/jk/trunk/native/common/jk_lb_worker.c Thu Jul 10 20:34:06 2014
@@ -1764,7 +1764,7 @@ static int JK_METHOD validate(jk_worker_
                     }
                     if (JK_IS_DEBUG_LEVEL(l)) {
                         jk_log(l, JK_LOG_DEBUG,
-                               "Balanced worker %i has name %s and route %s in 
domain %s",
+                               "Balanced worker %d has name %s and route %s in 
domain %s",
                                i,
                                p->lb_workers[i].name,
                                p->lb_workers[i].route,
@@ -1777,7 +1777,7 @@ static int JK_METHOD validate(jk_worker_
                     for (j = 0; j < i; j++) {
                         if (strcmp(p->lb_workers[i].route, 
p->lb_workers[j].route) == 0) {
                             jk_log(l, JK_LOG_ERROR,
-                                   "Balanced workers number %i (%s) and %i 
(%s) share the same route %s - aborting configuration!",
+                                   "Balanced workers number %d (%s) and %d 
(%s) share the same route %s - aborting configuration!",
                                    i,
                                    p->lb_workers[i].name,
                                    j,

Modified: tomcat/jk/trunk/native/common/jk_status.c
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/native/common/jk_status.c?rev=1609566&r1=1609565&r2=1609566&view=diff
==============================================================================
--- tomcat/jk/trunk/native/common/jk_status.c (original)
+++ tomcat/jk/trunk/native/common/jk_status.c Thu Jul 10 20:34:06 2014
@@ -3110,7 +3110,7 @@ static void commit_worker(jk_ws_service_
                        lb->retries, l);
     if (i != lb->retries && i > 0) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'retries' for lb worker '%s' to 
'%i'",
+               "Status worker '%s' setting 'retries' for lb worker '%s' to 
'%d'",
                w->name, name, i);
         lb->retries = i;
         sync_needed = JK_TRUE;
@@ -3119,7 +3119,7 @@ static void commit_worker(jk_ws_service_
                        lb->retry_interval, l);
     if (i != lb->retry_interval && i > 0) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'retry_interval' for lb worker '%s' 
to '%i'",
+               "Status worker '%s' setting 'retry_interval' for lb worker '%s' 
to '%d'",
                w->name, name, i);
         lb->retry_interval = i;
         sync_needed = JK_TRUE;
@@ -3128,7 +3128,7 @@ static void commit_worker(jk_ws_service_
                        lb->recover_wait_time, l);
     if (i != lb->recover_wait_time && i > 0) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'recover_time' for lb worker '%s' 
to '%i'",
+               "Status worker '%s' setting 'recover_time' for lb worker '%s' 
to '%d'",
                w->name, name, i);
         lb->recover_wait_time = i;
         sync_needed = JK_TRUE;
@@ -3137,7 +3137,7 @@ static void commit_worker(jk_ws_service_
                        lb->error_escalation_time, l);
     if (i != lb->error_escalation_time && i > 0) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'error_escalation_time' for lb 
worker '%s' to '%i'",
+               "Status worker '%s' setting 'error_escalation_time' for lb 
worker '%s' to '%d'",
                w->name, name, i);
         lb->error_escalation_time = i;
         sync_needed = JK_TRUE;
@@ -3146,7 +3146,7 @@ static void commit_worker(jk_ws_service_
                        lb->max_reply_timeouts, l);
     if (i != lb->max_reply_timeouts && i >= 0) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'max_reply_timeouts' for lb worker 
'%s' to '%i'",
+               "Status worker '%s' setting 'max_reply_timeouts' for lb worker 
'%s' to '%d'",
                w->name, name, i);
         lb->max_reply_timeouts = i;
         sync_needed = JK_TRUE;
@@ -3154,7 +3154,7 @@ static void commit_worker(jk_ws_service_
     i = status_get_bool(p, JK_STATUS_ARG_LB_STICKY, lb->sticky_session, l);
     if (i != lb->sticky_session) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'sticky_session' for lb worker '%s' 
to '%i'",
+               "Status worker '%s' setting 'sticky_session' for lb worker '%s' 
to '%d'",
                w->name, name, i);
         lb->sticky_session = i;
         sync_needed = JK_TRUE;
@@ -3162,7 +3162,7 @@ static void commit_worker(jk_ws_service_
     i = status_get_bool(p, JK_STATUS_ARG_LB_STICKY_FORCE, 
lb->sticky_session_force, l);
     if (i != lb->sticky_session_force) {
         jk_log(l, JK_LOG_INFO,
-               "Status worker '%s' setting 'sticky_session_force' for lb 
worker '%s' to '%i'",
+               "Status worker '%s' setting 'sticky_session_force' for lb 
worker '%s' to '%d'",
                w->name, name, i);
         lb->sticky_session_force = i;
         sync_needed = JK_TRUE;
@@ -3209,11 +3209,11 @@ static int set_int_if_changed(status_end
     if (i != *param && i >= min && i <= max) {
         if (lb_name)
             jk_log(l, JK_LOG_INFO,
-                   "Status worker '%s' setting '%s' for sub worker '%s' of lb 
worker '%s' to '%i'",
+                   "Status worker '%s' setting '%s' for sub worker '%s' of lb 
worker '%s' to '%d'",
                    w->name, att, name, lb_name, i);
         else
             jk_log(l, JK_LOG_INFO,
-                   "Status worker '%s' setting '%s' for ajp worker '%s' to 
'%i'",
+                   "Status worker '%s' setting '%s' for ajp worker '%s' to 
'%d'",
                    w->name, att, name, i);
         *param = i;
         return JK_TRUE;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to