Skip to content

Conversation

@bobtista
Copy link

Fixing typos in code comments.

@greptile-apps
Copy link

greptile-apps bot commented Jan 14, 2026

Greptile Summary

This PR corrects spelling errors across 79 files in code comments throughout Core/GameEngine and Core/Tools directories, with no functional code changes.

  • Fixed common misspellings: "occurances/occurance" → "occurrences/occurrence", "formated" → "formatted", "occured" → "occurred"
  • Corrected additional typos: "non-existant" → "non-existent", "visbility" → "visibility", "accomodate" → "accommodate", "succesfully" → "successfully", "independant" → "independent", "recognise" → "recognize", "Reutrn" → "Return"
  • Changes are documentation-only (comments), with zero impact on compiled code or runtime behavior
  • All 114 corrections reviewed are valid spelling fixes

Confidence Score: 5/5

  • This PR is completely safe to merge with zero risk
  • All changes are comment-only spelling corrections with no code modifications, making this a zero-risk documentation improvement with no possibility of introducing bugs or breaking changes
  • No files require special attention

Important Files Changed

Filename Overview
Core/GameEngine/Include/Common/AsciiString.h Fixed "occurances" to "occurrences" in comment
Core/GameEngine/Include/Common/file.h Fixed "formated" to "formatted" in comment
Core/GameEngine/Source/Common/System/AsciiString.cpp Fixed "occurances" to "occurrences" in comment
Core/GameEngineDevice/Source/W3DDevice/GameClient/BaseHeightMap.cpp Fixed "in almost" to "is almost" and "visbility" to "visibility" in comments
Core/Tools/Autorun/WSYS_file.h Fixed "occured" to "occurred" (3 instances) and "formated" to "formatted" in comments
Core/Tools/W3DView/Utils.cpp Fixed "occurance" to "occurrence" (2 instances) and "Reutrn" to "Return" in comments
Core/Tools/WW3D/pluglib/vector2.h Fixed "Subract" to "Subtract" and "independant" to "independent" in comments
Core/Tools/WW3D/pluglib/Vector.h Fixed "accomodate" to "accommodate" and "succesfully" to "successfully" in comments
Core/Tools/Launcher/Toolkit/Support/UString.cpp Fixed "occurance" to "occurrence" (4 instances) in function header comments
Core/Tools/wolSetup/WOLAPI/chatdefs.h Fixed "non-existant" to "non-existent" (2 instances), "occured" to "occurred", and "recognise" to "recognize" in comments

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant PR as Pull Request #2117
    participant Comments as Code Comments
    participant Codebase as Codebase
    
    Note over Dev,Codebase: Documentation-Only Changes (No Runtime Impact)
    
    Dev->>Comments: Identify spelling errors
    Note over Comments: occurances → occurrences<br/>formated → formatted<br/>occured → occurred<br/>non-existant → non-existent<br/>visbility → visibility<br/>and more...
    
    Dev->>PR: Fix 114 spelling errors in 79 files
    Note over PR: Core/GameEngine comments<br/>Core/Tools comments<br/>Header and source files
    
    PR->>Comments: Update comment text only
    Comments->>Codebase: Improved documentation clarity
    
    Note over Codebase: No functional changes<br/>No binary impact<br/>Safe to merge
Loading

@bobtista bobtista force-pushed the bobtista/typos-comments-core-other branch from fcc8772 to 568d5ca Compare January 14, 2026 22:14
@xezon xezon added Minor Severity: Minor < Major < Critical < Blocker Refactor Edits the code with insignificant behavior changes, is never user facing labels Jan 15, 2026
Copy link

@xezon xezon left a comment

Choose a reason for hiding this comment

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

Perfection again

@xezon xezon changed the title fix(typos): Fix spelling errors in Core/Tools and Core/GameEngine comments docs: Fix spelling errors in Core/Tools and Core/GameEngine comments Jan 16, 2026
@xezon xezon added Documentation Is documentation or complementary resource and removed Minor Severity: Minor < Major < Critical < Blocker Refactor Edits the code with insignificant behavior changes, is never user facing labels Jan 16, 2026
@xezon xezon merged commit e80bb51 into TheSuperHackers:main Jan 17, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Is documentation or complementary resource

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants