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

Fix binaryplatforms union invalidation #39251

Merged
merged 1 commit into from
Jan 14, 2021
Merged

Fix binaryplatforms union invalidation #39251

merged 1 commit into from
Jan 14, 2021

Conversation

timholy
Copy link
Sponsor Member

@timholy timholy commented Jan 14, 2021

DataStructures' union(s::DataStructures.SparseIntSet, ns) yields
a bad invalidation.

DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.
@timholy timholy added the backport 1.6 Change should be backported to release-1.6 label Jan 14, 2021
@timholy
Copy link
Sponsor Member Author

timholy commented Jan 14, 2021

(Backport only if it's easy & clean.)

Copy link
Member

@Sacha0 Sacha0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! :)

@staticfloat staticfloat merged commit 77865f2 into master Jan 14, 2021
@staticfloat staticfloat deleted the teh/binplat_invs branch January 14, 2021 22:51
@KristofferC KristofferC mentioned this pull request Jan 19, 2021
60 tasks
KristofferC pushed a commit that referenced this pull request Jan 19, 2021
DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.

(cherry picked from commit 77865f2)
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Feb 1, 2021
KristofferC pushed a commit that referenced this pull request Feb 1, 2021
DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.

(cherry picked from commit 77865f2)
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
DataStructures' `union(s::DataStructures.SparseIntSet, ns)` yields
a bad invalidation.

(cherry picked from commit 77865f2)
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.

4 participants