Skip to content

[Reflection] Fix iterateAsyncTaskAllocations. #38203

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

Merged
merged 1 commit into from
Jul 3, 2021

Conversation

mikeash
Copy link
Contributor

@mikeash mikeash commented Jul 1, 2021

Fix the declaration of AsyncTask and add a test for iterateAsyncTaskAllocations. Reflection's declaration of AsyncTask had fallen out of sync with the real thing. The test that was supposed to catch this was never actually committed, oops.

rdar://80035307

@mikeash mikeash requested review from al45tair and tbkka July 1, 2021 18:23
@mikeash mikeash force-pushed the fix-iterate-async-task-allocations branch from cc9df8b to f0e41d7 Compare July 1, 2021 19:25
@mikeash mikeash requested a review from DougGregor July 1, 2021 19:26
@mikeash
Copy link
Contributor Author

mikeash commented Jul 1, 2021

@swift-ci please test

@swift-ci
Copy link
Contributor

swift-ci commented Jul 1, 2021

Build failed
Swift Test Linux Platform
Git Sha - f0e41d7414ae943e156e68664e08620cab74c723

@swift-ci
Copy link
Contributor

swift-ci commented Jul 1, 2021

Build failed
Swift Test OS X Platform
Git Sha - f0e41d7414ae943e156e68664e08620cab74c723

@mikeash mikeash force-pushed the fix-iterate-async-task-allocations branch from f0e41d7 to 4e5655a Compare July 2, 2021 13:59
@mikeash
Copy link
Contributor Author

mikeash commented Jul 2, 2021

@swift-ci please test

@mikeash mikeash force-pushed the fix-iterate-async-task-allocations branch from 4e5655a to 56d8c31 Compare July 2, 2021 17:58
@mikeash
Copy link
Contributor Author

mikeash commented Jul 2, 2021

@swift-ci please test

Fix the declaration of AsyncTask and add a test for iterateAsyncTaskAllocations. Reflection's declaration of AsyncTask had fallen out of sync with the real thing. The test that was supposed to catch this was never actually committed, oops.

Add a swift_reflection_libraryVersion variable to Remote Mirror to indicate the presence of this fix. In the future, the value can be incremented to signal the presence of other changes that can't otherwise be detected.

rdar://80035307
@mikeash mikeash force-pushed the fix-iterate-async-task-allocations branch from 56d8c31 to c0e6e1d Compare July 2, 2021 21:33
@mikeash
Copy link
Contributor Author

mikeash commented Jul 2, 2021

@swift-ci please test

@mikeash mikeash merged commit 8cd6206 into swiftlang:main Jul 3, 2021
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.

2 participants