Documentation missing

Open Beginner friendly
#1,055 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Stale
Tech stack
csharp
Domain
documentation

Research direction

Open xml/System/String.xml and locate the String.GetHashCode(StringComparison) entry, then compare it with the corresponding source in System.Private.CoreLib/shared/System/String.Comparison.cs. Update the API documentation with guidance on when this overload should be used, and verify the rendered String.GetHashCode reference page contains the new explanation.

Written by the indexing model from the issue text.

Description

area-System.Runtime dotnet-api/prod Pri2 untriaged

The overloaded method public int GetHashCode(StringComparison comparisonType) is not documented. Its is only mentioned it here and there is no guidance when it should be used. The source file says:

> Gets a hash code for this string and this comparison. If strings A and B and comparison C are such that string.Equals(A, B, C), then they will return the same hash code with this comparison C.

See: https://github.com/dotnet/coreclr/blob/master/src/System.Private.CoreLib/shared/System/String.Comparison.cs.


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Dominant language
C#
Stars
950
Forks
1.7k
Avg merge
3d 5h
Merged PRs (30d)
38

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dotnet/dotnet-api-docs

All issues in dotnet/dotnet-api-docs

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.