On 16/11/2010 18:35, sebb wrote: > On 16 November 2010 18:07, <ma...@apache.org> wrote: >> Author: markt >> Date: Tue Nov 16 18:07:56 2010 >> New Revision: 1035720 >> >> URL: http://svn.apache.org/viewvc?rev=1035720&view=rev >> Log: >> Session manager performance >> Move sync. No performance change but sync requirements for sub-classes are >> clearer. >> Still looking to remove sync completely.
> Does the method pre-amble (as above) need to be synch? > AFAICT, most other accesses to devRandomSource and randomIS within the > module are not protected by synch. Maybe. There may be issues around changing devRandomSource whilst the application is running. That isn't the focus of what I am looking at right now (and I am not sure that should be supported anyway) but I am keeping it in mind. My intention is to reduce the scope of the sync as far as possible and ideally remove it. I am doing this via small incremental changes rather than one big commit so folks can follow what I am doing and how I am doing it. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
