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

RangeSet should indicate if a mutation operation was a no-op #1928

Open
dimo414 opened this issue Dec 23, 2014 · 0 comments
Open

RangeSet should indicate if a mutation operation was a no-op #1928

dimo414 opened this issue Dec 23, 2014 · 0 comments
Labels
P4 package=collect type=enhancement Make an existing feature better

Comments

@dimo414
Copy link
Contributor

dimo414 commented Dec 23, 2014

RangeSet's mutation methods (add(), remove(), etc.) could leave the RangeSet unchanged (if rangeSet.encloses(range) or range.isEmpty()) but there's no easy way to detect when this occurs without manually inspecting the RangeSet or Range before the operation.

The equivalent methods on Set all return a boolean, returning true if the Set changed as a result of the operation; it would be useful if RangeSet behaved similarly.

@nick-someone nick-someone added P4 type=enhancement Make an existing feature better labels Aug 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P4 package=collect type=enhancement Make an existing feature better
Projects
None yet
Development

No branches or pull requests

3 participants