Skip to content

Fix duplicated-word and misspelled comment/doc typos#2092

Closed
durvesh1992 wants to merge 1 commit into
facebook:static_hfrom
durvesh1992:fix/comment-doc-typos
Closed

Fix duplicated-word and misspelled comment/doc typos#2092
durvesh1992 wants to merge 1 commit into
facebook:static_hfrom
durvesh1992:fix/comment-doc-typos

Conversation

@durvesh1992

Copy link
Copy Markdown

Summary

Fixes comment-only and documentation typos found via an inspection-only sweep of the codebase (dual pass: duplicated-word regex + common-misspelling wordlist, each hit manually confirmed). No code logic is changed — every edit is inside a ////// comment, a block comment, or a Markdown doc.

I deliberately excluded candidates already covered by the open typo PRs #1976 and #1656, as well as enviroment (#1900). Generated/vendored trees (external/, third-party/, benchmarks/ vendored JS) and test CHECK fixtures were skipped.

Categories

  • Duplicated words (the the, with with, for for, that that, from from, in in, we we, and and, but but, It it, If if): 18 fixes
  • Misspellings (Overriden->Overridden ×3, funciton->function ×2, Supress->Suppress, sucessfully->successfully, Wether->Whether): 7 fixes

Before / After (sample)

File:Line Before After
include/hermes/Regex/Regex.h:88 Construct and and append Construct and append
include/hermes/VM/ArrayStorage.h:54 \return The the maximum \return The maximum
include/hermes/VM/ModuleExportsCache.h:26 If if \p modIndex If \p modIndex
lib/Support/BigIntSupport.cpp:410 filled with with the most filled with the most
lib/Support/OSCompatPosix.cpp:502 number from from statm number from statm
lib/VM/DictPropertyMap.cpp:174 It it happens It happens
doc/IR.md:217,228,238,248 The the instruction The instruction
include/hermes/BCGen/HBC/BCProvider.h:278 in funciton in function
include/hermes/Support/SourceErrorManager.h:171 Supress errors Suppress errors
include/hermes/Support/SourceErrorManager.h:476 If sucessfully decoded If successfully decoded
API/hermes/CompileJS.h:48 Wether to generate Whether to generate
tools/hermes-parser/.../HermesASTAdapter.js:76,107 Overriden in child classes Overridden in child classes

Full set: 25 single-token fixes across 18 files (git diff --stat: 25 insertions, 25 deletions).

Fixes comment-only and documentation typos found via an inspection sweep:
duplicated words ("the the", "with with", "for for", "that that", etc.) and
common misspellings (Overriden -> Overridden, funciton -> function,
Supress -> Suppress, sucessfully -> successfully, Wether -> Whether).

These are comment/doc-comment/markdown changes only; no code logic is affected.
Candidates already covered by other open typo PRs were excluded.
@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 29, 2026
@avp

avp commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Sorry, we don't take typo PRs on this repo.

@avp avp closed this Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants