Skip to content

Length parameter description confusing for constructors with sbyte* #11136

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

Open
rindlespot opened this issue Mar 28, 2025 · 1 comment
Open
Labels
area-System.Runtime help wanted Good for community contributors to help [up-for-grabs] untriaged New issue has not been triaged by the area owner

Comments

@rindlespot
Copy link

Type of issue

Typo

Description

There are two constructors on this page which take an sbyte* input parameter. For each, the length parameter is described as:

The number of characters within value to use.

Unlike the other constructors on this page, using the term 'characters' for these two is misleading. The number of characters within an sbyte* is variable, depending on how the decoding works out. Fortunately, what this parameter really wants is a count of the number of sbytes, not the (more difficult to calculate) number of characters.

Proposed replacement text:

The number of sbytes within value to use.

Page URL

https://learn.microsoft.com/en-us/dotnet/api/system.string.-ctor?view=net-9.0

Content source URL

https://github.com/dotnet/dotnet-api-docs/blob/main/xml/System/String.xml

Document Version Independent Id

90baab9e-9ef3-9bb5-d433-8d861d7fa876

Platform Id

ac6a36d8-9684-17ab-5b56-e2696ccb8adf

Article author

@dotnet-bot

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Mar 28, 2025
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Mar 28, 2025
@jozkee jozkee added area-System.Runtime and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Mar 28, 2025
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-runtime

@jozkee jozkee added the help wanted Good for community contributors to help [up-for-grabs] label Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Runtime help wanted Good for community contributors to help [up-for-grabs] untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

2 participants