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.

Sign up for GitHub

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

Jump to bottom

PollLast might not work properly (needs confirmation) #23

Closed
Speiger opened this issue Dec 4, 2022 · 1 comment
Closed

PollLast might not work properly (needs confirmation) #23

Speiger opened this issue Dec 4, 2022 · 1 comment
Labels
invalid This doesn't seem right

Comments

@Speiger
Copy link
Owner

Speiger commented Dec 4, 2022

As Reported to FastUtil by @Z-Kris FastUtils HashSet removeLast (or pollLast in this library) has a issue with iterators afterwards.
I am pretty sure I caught that in my Unit test implementations but I would rather double check this myself, to make sure it is fixed.

If it is a issue this will be addressed with the 0.7.1 or 0.8.0 patch (the next patch might not be 0.7.1, but 0.8.0).

Also @Z-Kris thanks for the report.
@Speiger Speiger added the invalid This doesn't seem right label Dec 13, 2022
@Speiger
Copy link
Owner Author

Speiger commented Dec 13, 2022

Checked the code, this wasn't a bug with Primitive Collections

@Speiger Speiger closed this as completed Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Speiger