Skip to content

Spelling stdlib/public/concurrency #42443

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 15 commits into from
Apr 21, 2022

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Apr 19, 2022

This fixes some misspellings in Swift stdlib/public/concurrency, it is split per #42421 (comment)

jsoref added 15 commits April 19, 2022 00:12
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
@jsoref jsoref closed this Apr 19, 2022
@jsoref jsoref deleted the spelling-stdlib-public-concurrency branch April 19, 2022 18:18
@jsoref jsoref restored the spelling-stdlib-public-concurrency branch April 19, 2022 18:19
@jsoref jsoref reopened this Apr 19, 2022
@CodaFi CodaFi requested a review from amartini51 April 19, 2022 18:42
@CodaFi
Copy link
Contributor

CodaFi commented Apr 19, 2022

@swift-ci smoke test

@@ -106,15 +106,15 @@ FutureFragment::Status AsyncTask::waitFuture(AsyncTask *waitingTask,
auto fragment = futureFragment();

auto queueHead = fragment->waitQueue.load(std::memory_order_acquire);
bool contextIntialized = false;
bool contextInitialized = false;
Copy link
Member

Choose a reason for hiding this comment

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

This is a change to the code, not just to docs or comments, so someone other than me should review it.

@amartini51
Copy link
Member

Overall, these changes look correct to me. I've flagged one place where the part of the actual code is being changed.

@tshortli tshortli merged commit 7a57e7c into swiftlang:main Apr 21, 2022
@jsoref jsoref deleted the spelling-stdlib-public-concurrency branch April 21, 2022 18:14
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