Greetings All. The following link is a patch which I believe simplifies the structure of the _sevice() function by removing some NULL tests and an If(test) from the processing stream. http://www.gknw.com/norm/jk_worker_status.c.diff The following section shows the structure implemented rather the diff itself as is the usual case. By setting a default query_string first, the NULL tests are no loner required. By doing the common headers first, only one if(qry=) test is needed. Norm -------------------------------------- if( w->mbean->debug > 0 ) env->l->jkLog(env, env->l, JK_LOG_DEBUG, "status.service() %s %s\n", JK_CHECK_NULL(uri), JK_CHECK_NULL(s->query_string));
if( s->query_string == NULL ) { s->query_string="all"; } /* Generate the header */ s->status=200; s->msg="OK"; s->headers_out->put(env, s->headers_out, "Pragma", "no-cache", NULL); s->headers_out->put(env, s->headers_out, "Cache-Control", "no-cache", NULL); /** Decide if text or html. */ if( strncmp( s->query_string, "qry=", 4) == 0 ) { s->headers_out->put(env, s->headers_out, "Content-Type", "text/plain", NULL); s->head(env, s ); } else { s->headers_out->put(env, s->headers_out, "Content-Type", "text/html", NULL); s->head(env, s ); s->jkprintf(env, s, "<style>%s</style>\n", DEFAULT_CSS ); } /** Process the query string. */ -------------------------------------- --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]