Skip to content

Correct the Xcode fixits for dispatch_async() and dispatch_group_async() #205

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
Jan 27, 2017
Merged

Correct the Xcode fixits for dispatch_async() and dispatch_group_async() #205

merged 1 commit into from
Jan 27, 2017

Conversation

ktopley-apple
Copy link
Contributor

@ktopley-apple ktopley-apple commented Jan 25, 2017

The same change was already made in the swift repo (swiftlang/swift#6958)

rdar://problem/27915073

@ktopley-apple
Copy link
Contributor Author

@swift-ci Please test

Copy link
Contributor

@moiseev moiseev left a comment

Choose a reason for hiding this comment

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

:shipit:

@ktopley-apple
Copy link
Contributor Author

Test failure is unrelated to this change.

@ktopley-apple
Copy link
Contributor Author

@swift-ci Please test

@ktopley-apple
Copy link
Contributor Author

@dgrove-oss could you please take a look at the build output for Linux? It's failing with a timeout in the SwiftPM tests.

@dgrove-oss
Copy link
Contributor

I think there is something broken in how the libdispatch PR testing is being done. It has been habitually failing on this swiftPM subtest (and we have been ignoring the failure, merging the changes, and not seeing the failure in the main runs). I'll ask for help on swift-dev list to see if anyone has an idea of what could be wrong.

@ktopley-apple
Copy link
Contributor Author

@dgrove-oss thanks. The failure is clearly not related to the change, so I am going to go ahead and merge.

@ktopley-apple ktopley-apple merged commit b6e9cf4 into swiftlang:master Jan 27, 2017
das pushed a commit that referenced this pull request Feb 21, 2017
Correct the Xcode fixits for dispatch_async() and dispatch_group_async()

Signed-off-by: Daniel A. Steffen <dsteffen@apple.com>
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