I see you've checked in the iterator -> int fix, thank you both.

Peter wrote: "revert the script test that showed up the problem in the first
place". Sorry, but what does this mean to the non-antdev-initiated? I am
using Krysalis Centipede, which is clearly too big to have as a unit test
for ant, but which does gump off nightly ant, and will test this feature.
You can (at the least) count on us to report any failures we get to the ant
list. :-) Do you need us to attempt to create a unit test for ant, or is
this enough?

Again, thanks for your quick work and support with this fix.

regards

Adam
-----Original Message-----
From: peter reilly [mailto:[EMAIL PROTECTED]
Sent: Tuesday, July 15, 2003 2:54 AM
To: Ant Developers List
Subject: Re: FW: ConcurrentModificationException


Yep, the import task works again.

We probably should revert the script
test that showed up the problem in the first
place.

Peter

On Tue, 2003-07-15 at 09:35, Stefan Bodewig wrote:
> With a loop like
>
>             for (int i = 0; i < children.size(); i++) {
>                 Object o = children.get(i);
>
> instead of
>
>             Iterator it = children.iterator();
>             while (it.hasNext()) {
>                 Object o = it.next();
>
> we should be on the save side, I think.  children can only grow as
> there are only methods to add, not to remove tasks/types.
>
> Stefan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to