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.

Sign up for GitHub

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

Jump to bottom

spelling mistake in System/Collections/Generic/LinkedList.cs line 180 #55389

Closed
RunningDeer opened this issue Jul 9, 2021 · 2 comments · Fixed by #55634
Closed

spelling mistake in System/Collections/Generic/LinkedList.cs line 180 #55389

RunningDeer opened this issue Jul 9, 2021 · 2 comments · Fixed by #55634
Labels
area-System.Collections documentation Documentation bug or enhancement, does not impact product or test code
Milestone
6.0.0

Comments

@RunningDeer
Copy link

dotnet/runtime/blob/main/src/libraries/System.Collections/src/System/Collections/Generic/LinkedList.cs line 180

the comment is unclear, seems to be superfluous 'the'.
@dotnet-issue-labeler dotnet-issue-labeler bot added area-System.Collections untriaged New issue has not been triaged by the area owner labels Jul 9, 2021
@ghost
Copy link

ghost commented Jul 9, 2021

Tagging subscribers to this area: @eiriktsarpalis
See info in area-owners.md if you want to be subscribed.

Issue Details

dotnet/runtime/blob/main/src/libraries/System.Collections/src/System/Collections/Generic/LinkedList.cs line 180

the comment is unclear, seems to be superfluous 'the'.

Author: RunningDeer
Assignees: -
Labels:

area-System.Collections, untriaged
Milestone: -

@GrabYourPitchforks
Copy link
Member

Thought this might be referring to some code that was previously deleted, but checked the Whidbey sources (Jun 2005) and saw that the comment was still there verbatim. Agree that it seems to be erroneous. I think we should just remove the entire comment.

eiriktsarpalis added a commit that referenced this issue Jul 14, 2021
@eiriktsarpalis
remove superfluous comment
8f0abb4 
Fix #55389
@eiriktsarpalis eiriktsarpalis mentioned this issue Jul 14, 2021
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 14, 2021
@eiriktsarpalis eiriktsarpalis added documentation Documentation bug or enhancement, does not impact product or test code and removed untriaged New issue has not been triaged by the area owner labels Jul 14, 2021
@eiriktsarpalis eiriktsarpalis added this to the 6.0.0 milestone Jul 14, 2021
stephentoub pushed a commit that referenced this issue Jul 14, 2021
@eiriktsarpalis
remove superfluous comment (#55634)
0a42d49 
Fix #55389
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 14, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Aug 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-System.Collections documentation Documentation bug or enhancement, does not impact product or test code
Projects
None yet
Milestone
6.0.0
Development

Successfully merging a pull request may close this issue.

Remove superfluous comment dotnet/runtime
3 participants
@GrabYourPitchforks @eiriktsarpalis @RunningDeer