Author: mturk Date: Mon Apr 2 00:14:47 2007 New Revision: 524737 URL: http://svn.apache.org/viewvc?view=rev&rev=524737 Log: Fix LF/CRLF logging for POSIX/WIN32 platforms
Modified: tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c tomcat/connectors/trunk/jk/native/common/jk_util.c Modified: tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c?view=diff&rev=524737&r1=524736&r2=524737 ============================================================================== --- tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c (original) +++ tomcat/connectors/trunk/jk/native/apache-1.3/mod_jk.c Mon Apr 2 00:14:47 2007 @@ -2370,11 +2370,19 @@ int log_fd = flp->log_fd; size_t sz = strlen(what); if (log_fd >= 0 && sz) { - if ( write(log_fd, what, sz) < 0 ) - { + if (write(log_fd, what, sz) < 0 ) { ap_log_error(APLOG_MARK, APLOG_ERR | APLOG_NOERRNO, NULL, "mod_jk: jk_log_to_file %s failed", what); + } + else { + char c; +#if defined(WIN32) + c = '\n'; + write(log_fd, &c, 1); +#endif + c = '\n'; + write(log_fd, &c, 1); } } Modified: tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c?view=diff&rev=524737&r1=524736&r2=524737 ============================================================================== --- tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c (original) +++ tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c Mon Apr 2 00:14:47 2007 @@ -2516,6 +2516,10 @@ "mod_jk: jk_log_to_file %s failed: %s", what, error); } +#if defined(WIN32) + apr_file_putc('\r', p->jklogfp); +#endif + apr_file_putc('\n', p->jklogfp); rv = apr_global_mutex_unlock(jk_log_lock); if (rv != APR_SUCCESS) { ap_log_error(APLOG_MARK, APLOG_ERR, rv, NULL, Modified: tomcat/connectors/trunk/jk/native/common/jk_util.c URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_util.c?view=diff&rev=524737&r1=524736&r2=524737 ============================================================================== --- tomcat/connectors/trunk/jk/native/common/jk_util.c (original) +++ tomcat/connectors/trunk/jk/native/common/jk_util.c Mon Apr 2 00:14:47 2007 @@ -400,6 +400,7 @@ jk_file_logger_t *p = l->logger_private; if (p->logfile) { fputs(what, p->logfile); + fputc('\n', p->logfile); /* [V] Flush the dam' thing! */ fflush(p->logfile); } @@ -492,8 +493,8 @@ const char *fmt, ...) { int rc = 0; - /* Need to reserve space for newline and terminating zero byte. */ - static int usable_size = HUGE_BUFFER_SIZE-2; + /* Need to reserve space for terminating zero byte. */ + static int usable_size = HUGE_BUFFER_SIZE - 1; if (!l || !file || !fmt) { return -1; } @@ -576,7 +577,6 @@ } else { used = usable_size; } - buf[used++] = '\n'; buf[used] = 0; l->log(l, level, buf); #ifdef NETWARE --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]