If the worker thread synchronization is working properly, it's not
necessary to set these fields to NULL after removing an element from
the queue, and it's IMO more important to have a shorter code path
leading up to the mutex_unlock. I left in support for NULLing the
fields when debugging, though.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@94850
13f79535-47bb-0310-9956-
ffa450edef68
elem = &queue->data[--queue->nelts];
*sd = elem->sd;
*p = elem->p;
+#ifdef AP_DEBUG
elem->sd = NULL;
elem->p = NULL;
+#endif /* AP_DEBUG */
rv = apr_thread_mutex_unlock(queue->one_big_mutex);
return rv;