Skip to content

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


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