Skip to content

Update wctomb-s-wctomb-s-l.md - #4463

Merged
Jak Koke (Jak-MS) merged 3 commits into
MicrosoftDocs:mainfrom
EddieBreeveld:patch-3
Mar 16, 2023
Merged

Jak Koke (Jak-MS) merged 3 commits into
MicrosoftDocs:mainfrom
EddieBreeveld:patch-3

Conversation

@EddieBreeveld

Copy link
Copy Markdown
Contributor

Adding another possible error return. I'm not sure of the document syntax though.

Adding another possible error return.  I'm not sure of the document syntax though.
@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7824880:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/wctomb-s-wctomb-s-l.md ✅Succeeded

For more details, please refer to the build report.

Note: Links can become broken if there are changes on the target sites.

For any questions, please:

@Court72

Copy link
Copy Markdown
Contributor

Tyler Whitney (@TylerMSFT)

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

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

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged Tracking label for the PR review team label Mar 15, 2023
If any of the above error conditions occurs, the invalid parameter handler is invoked, as described in [Parameter validation](../parameter-validation.md). If execution is allowed to continue, `wctomb` returns `EINVAL` and sets `errno` to `EINVAL`.

Return value `EILSEQ` indicates an encoding error has occurred.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for making this addition. Could we word it slightly differently and provide a link? For example: "An EILSEQ return value indicates that the parameter wchar is not a valid wide character.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The link may be hard to read since github interpreted the markdown. It was [EILSEQ](../errno-constants.md)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

with backticks around EILSEQ

@TylerMSFT Tyler Whitney (TylerMSFT) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for improving the docs! What do you think of the suggestion I made in the file.

Added code escape so machine translation doesn't modify EILSEQ
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ec8a273:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/wctomb-s-wctomb-s-l.md ✅Succeeded

For more details, please refer to the build report.

Note: Links can become broken if there are changes on the target sites.

For any questions, please:

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit bd89f36:

✅ Validation status: passed

File Status Preview URL Details
docs/c-runtime-library/reference/wctomb-s-wctomb-s-l.md ✅Succeeded

For more details, please refer to the build report.

Note: Links can become broken if there are changes on the target sites.

For any questions, please:

@TylerMSFT

Copy link
Copy Markdown
Collaborator

#sign-off

@Jak-MS
Jak Koke (Jak-MS) merged commit 8b98497 into MicrosoftDocs:main Mar 16, 2023
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.

4 participants