Re: svn commit: r564424 - /xml/xindice/trunk/java/src/org/apache/xindice/core/indexer/

On 8/9/07, Vadim Gritsenko <[email protected]> wrote:
> [email protected] wrote:
> > @@ -175,12 +163,9 @@
> >       * Drop all indexers
> >       */
> >      public synchronized void drop() {
> > -        // Get a copy of idxList
> > -        IndexerInfo[] idx = idxList;
> > -
> >          // Drop indexes
> > -        for (int i = 0; i < idx.length; i++) {
> > -            drop(idx[i].name);
> > +        for (Iterator i = indexes.keySet().iterator(); i.hasNext(); ) {
> > +            drop((String) i.next());
> >          }
> >      }
>
> This causes concurrent modification exception: call to drop(String) modifies
> indexes collection.

Oops... Thanks.

> Hmm, test suite needs some improvement ;)

I am not sure that there should be a testcase for that specific issue :)

Natalia