Skip to content

Update multithreading-and-locales.md #4078

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
Aug 4, 2022
Merged

Conversation

Antonz0
Copy link
Contributor

@Antonz0 Antonz0 commented Aug 4, 2022

Wrong setlocale() reference, it should be a C runtime function, not a preprocessor macro.

Wrong `setlocale()` reference, it should be a C runtime function, not a preprocessor macro.
@ghost
Copy link

ghost commented Aug 4, 2022

CLA assistant check
All CLA requirements met.

@PRMerger10
Copy link
Contributor

@Antonz0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld32
Copy link

opbld32 commented Aug 4, 2022

Docs Build status updates of commit 13d99b1:

✅ Validation status: passed

File Status Preview URL Details
docs/parallel/multithreading-and-locales.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Aug 4, 2022

@TylerMSFT

Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@PRMerger15 PRMerger15 added the aq-pr-triaged Tracking label for the PR review team label Aug 4, 2022
Copy link
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

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

At least we were consistent. This change looks good to me. Thanks, @Antonz0

@colin-home
Copy link
Contributor

#sign-off

@PRMerger18
Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @TylerMSFT

@colin-home colin-home merged commit 67ba638 into MicrosoftDocs:main Aug 4, 2022
@opbld31
Copy link

opbld31 commented Aug 4, 2022

Docs Build status updates of commit 13d99b1:

✅ Validation status: passed

File Status Preview URL Details
docs/parallel/multithreading-and-locales.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.