Although it took two commits, please comment on the following patch to eliminate an 'unused variable' rv in some cases (happy to back out if there are objections);
Index: server/core.c^M =================================================================== --- server/core.c (revision 292396) +++ server/core.c (working copy) @@ -4485,9 +4485,10 @@ static int core_pre_connection(conn_rec *c, void *csd) { core_net_rec *net = apr_palloc(c->pool, sizeof(*net)); + +#ifdef AP_MPM_DISABLE_NAGLE_ACCEPTED_SOCK apr_status_t rv; -#ifdef AP_MPM_DISABLE_NAGLE_ACCEPTED_SOCK /* BillS says perhaps this should be moved to the MPMs. Some OSes * allow listening socket attributes to be inherited by the * accept sockets which means this call only needs to be made Thanks, Bill