sebb wrote: > If it is intended to allow currentDate to be updated by another thread > before the return, then the field needs to be volatile. Otherwise the > return value needs to be established in the synch. block.
It needs to be volatile. I ran some comparisons and the performance difference appears to be in the noise. Thanks for the review. I'll also see what changes, if any, this implies for the AccessLogValve. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org