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

Vadim Gritsenko <[email protected]>
Newsgroups gmane.text.xml.xindice.devel
Message-ID <[email protected]>
[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.

Hmm, test suite needs some improvement ;)

Vadim
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.