Missread the diff. Thanks.

Todd

On 10/26/06, Vadim Gritsenko <[EMAIL PROTECTED]> wrote:
Todd Byrne wrote:
> In the DatabaseShudownHandler.run method the synchronized block needs to
> be outside the loop. Its possible for the Map to be modified by a
> different thread and that will throw a ConcurrentModificationExpection
> from the iteration.

Yep, it is there:

>     +        synchronized (databases) {
>     +            Iterator dbs = databases.iterator();


Vadim

Reply via email to