Skip to content
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

UintPtr & IntPtr struct – rewording #7209

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

deep-outcome
Copy link
Contributor

Summary

Microfix of bad wording.

@opbld32
Copy link

opbld32 commented Sep 19, 2021

Docs Build status updates of commit b655649:

✅ Validation status: passed

File Status Preview URL Details
xml/System/IntPtr.xml ✅Succeeded View
xml/System/UIntPtr.xml ✅Succeeded View

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:

@MSDN-WhiteKnight
Copy link
Contributor

Why is this change needed? Is the current wording incorrect grammar, unclear, or considered bad writing style?

@deep-outcome
Copy link
Contributor Author

Why is this change needed? Is the current wording incorrect grammar, unclear, or considered bad writing style?

It has bad formula. Data are not referenced by language. At least for me, it seems confusing. The more that UintPtr/IntPtr will likely be used only in one of them. So between as conjunction of meaning in the middle is slightly inaccurate.

Sense is clear in common language but in technical vocabulary I mean not.

@MSDN-WhiteKnight
Copy link
Contributor

This PR likely conflicted with #8132

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.

3 participants