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

Reply via email to