Skip to content

chore: remove repetitive words in comment - #90759

Open
futurehua wants to merge 1 commit into
swiftlang:mainfrom
futurehua:main
Open

chore: remove repetitive words in comment#90759
futurehua wants to merge 1 commit into
swiftlang:mainfrom
futurehua:main

Conversation

@futurehua

Copy link
Copy Markdown

remove repetitive words in comment

@ktoso ktoso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@ktoso
ktoso enabled auto-merge July 17, 2026 09:24

@mikeash mikeash left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh. That's a lot more than I expected.

@rjmccall rjmccall left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"that that" is grammatical in both of these cases. The rest of these look fine.

Comment thread include/swift/ABI/TypeIdentity.h Outdated
Comment thread stdlib/public/Distributed/DistributedDefaultExecutor.swift Outdated

@al45tair al45tair left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two were incorrect, but just deleting the repeated word doesn't actually fix the sentence.

Comment thread stdlib/public/RuntimeModule/Mangling.swift Outdated
Comment thread stdlib/public/RuntimeModule/Mangling.swift Outdated
Signed-off-by: futurehua <futurehua@outlook.com>
auto-merge was automatically disabled July 18, 2026 02:46

Head branch was pushed to by a user without write access

@futurehua

Copy link
Copy Markdown
Author

@rjmccall @al45tair Thank you for your review. Modified.

@compnerd compnerd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The DocC change feels off - is that hand generated?

@futurehua

Copy link
Copy Markdown
Author

@glessard Thank you for your approval. If there's anything need I to do, please feel free to let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants