Skip to content

Don't remove elements from a map during iteration.#4018

Merged
lrytz merged 1 commit intoscala:2.11.xfrom
retronym:ticket/8774-concurrent-modification
Sep 30, 2014
Merged

Don't remove elements from a map during iteration.#4018
lrytz merged 1 commit intoscala:2.11.xfrom
retronym:ticket/8774-concurrent-modification

Conversation

@retronym
Copy link
Member

Doing so relies on implementation details which might change.

See #3911 / SI-8774.

Review by @lrytz; /cc @alextg-github

Doing so relies on implementation details which might change.

See scala#3911 / SI-8774.
@lrytz
Copy link
Member

lrytz commented Sep 30, 2014

LGTM. I agree that an eager ConcurrentModificationException is much better than unspecified behavior.

lrytz added a commit that referenced this pull request Sep 30, 2014
…ation

Don't remove elements from a map during iteration.
@lrytz lrytz merged commit ce7f9f0 into scala:2.11.x Sep 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants