Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make concurrent modification detection on Qt containers optional #153

Closed
omix opened this issue May 11, 2023 · 1 comment
Closed

Make concurrent modification detection on Qt containers optional #153

omix opened this issue May 11, 2023 · 1 comment
Assignees

Comments

@omix
Copy link
Contributor

omix commented May 11, 2023

QtJambi's iterators do a container modification check on each next()/hasHext().

QStringList stringlist = ...
for(String s : stringlist){
    stringlist.append("new string");
}
// will throw ConcurrentModificationException

This is time consuming and should be optional only.

@omix omix self-assigned this May 11, 2023
@omix omix changed the title Make concurrent modification detection on Qt containers optional Make modification detection on Qt containers optional May 11, 2023
@omix omix changed the title Make modification detection on Qt containers optional Make concurrent modification detection on Qt containers optional May 11, 2023
@omix
Copy link
Contributor Author

omix commented May 11, 2023

This feature will be available in QtJambi 6.5.1.

Runtime argument -Dio.qt.enable-concurrent-container-modification-check=true will activate concurrent modification checks during container iteration.

omix added a commit that referenced this issue May 12, 2023
issue #153
issue #152
issue #151
issue #150
issue #149
issue #148
issue #147
omix added a commit that referenced this issue May 25, 2023
issue #158
issue #153
issue #152
issue #151
issue #150
issue #149
issue #148
issue #147
@omix omix closed this as completed May 25, 2023
omix added a commit that referenced this issue Jul 25, 2023
issue #153
issue #152
issue #151
issue #150
issue #149
issue #148
issue #147
omix added a commit that referenced this issue Jul 25, 2023
issue #158
issue #153
issue #152
issue #151
issue #150
issue #149
issue #148
issue #147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant