-
Notifications
You must be signed in to change notification settings - Fork 967
Update wctomb-s-wctomb-s-l.md #4463
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
Conversation
Adding another possible error return. I'm not sure of the document syntax though.
@EddieBreeveld : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 7824880: ✅ Validation status: passed
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:
|
Can you review the proposed changes? When the changes are ready for publication, add a #label:"aq-pr-triaged" |
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. |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with backticks around EILSEQ
There was a problem hiding this comment.
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 a link.
Added code escape so machine translation doesn't modify EILSEQ
Learn Build status updates of commit ec8a273: ✅ Validation status: passed
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 status updates of commit bd89f36: ✅ Validation status: passed
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:
|
#sign-off |
Adding another possible error return. I'm not sure of the document syntax though.