-
Notifications
You must be signed in to change notification settings - Fork 966
Update wcstombs-s-wcstombs-s-l.md #3416
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
Update wcstombs-s-wcstombs-s-l.md #3416
Conversation
count should reserve one place for terminating null character.
@mohammad-ghasemi-2549 : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
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.
@mohammad-ghasemi-2549 , thank you for your interest in the docs. At first I wasn't clear about your change because The docs call out that the count
parameter specifically does not account for the NULL character. But I think what you are doing is accounting for that so that the trailing NULL doesn't land 1 past the buffer in case the text takes up the amount of room for the buffer. Yes?
yes |
Add a couple other fixes since the file is being updated.
#sign-off |
@mohammad-ghasemi-2549, could you try signing the CLA again above? I'd like to get your change merged. |
@TylerMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
The CLA signing problem appears to be resolved. Merging now. |
count parameter should reserve one place for the terminating null character.