Skip to content

fix(comments): Use 2026 for this year's code comments#2285

Merged
xezon merged 3 commits intoTheSuperHackers:mainfrom
Caball009:fix_comment_dates_TSH
Feb 10, 2026
Merged

fix(comments): Use 2026 for this year's code comments#2285
xezon merged 3 commits intoTheSuperHackers:mainfrom
Caball009:fix_comment_dates_TSH

Conversation

@Caball009
Copy link

@Caball009 Caball009 commented Feb 9, 2026

I noticed that in a couple of places there are 'TheSuperHackers' comments with a date that should use 2026 instead of 2025. This PR fixes that. I used the follow regex query: TheSuperHackers.*\d\d/0[12]/2025

Replicated in Generals in the second commit.

@Caball009 Caball009 added Minor Severity: Minor < Major < Critical < Blocker ThisProject The issue was introduced by this project, or this task is specific to this project Fix Is fixing something, but is not user facing labels Feb 9, 2026
@Caball009 Caball009 changed the title fix(comments): Use 2026 for this year's code code comments fix(comments): Use 2026 for this year's code comments Feb 9, 2026
@greptile-apps
Copy link

greptile-apps bot commented Feb 9, 2026

Greptile Overview

Greptile Summary

This PR updates several TheSuperHackers comment annotations across both the Generals and GeneralsMD trees, changing dates from 2025 to 2026. The edits are confined to comment lines in ActionManager.cpp, ReplayMenu.cpp, dx8caps.cpp, and render2d.cpp, with no functional code changes.

These files span gameplay logic (construction resume ownership checks), GUI replay list rendering, and WW3D2 DirectX capability/rendering paths; however, the PR does not alter any of that behavior—only the year in the existing comment metadata.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • All hunks are comment-only year updates (2025 -> 2026) with no functional code changes, header/prologue edits, or formatting shifts in code blocks.
  • No files require special attention.

Important Files Changed

Filename Overview
Generals/Code/GameEngine/Source/Common/RTS/ActionManager.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.
Generals/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/ReplayMenu.cpp Updates two TheSuperHackers comment dates from 2025 to 2026; no functional code changes.
Generals/Code/Libraries/Source/WWVegas/WW3D2/dx8caps.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.
Generals/Code/Libraries/Source/WWVegas/WW3D2/render2d.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.
GeneralsMD/Code/GameEngine/Source/Common/RTS/ActionManager.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.
GeneralsMD/Code/GameEngine/Source/GameClient/GUI/GUICallbacks/Menus/ReplayMenu.cpp Updates two TheSuperHackers comment dates from 2025 to 2026; no functional code changes.
GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dx8caps.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.
GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/render2d.cpp Updates a TheSuperHackers comment date from 2025 to 2026; no functional code changes.

Sequence Diagram

sequenceDiagram
  participant Dev as Developer
  participant Repo as Source Tree
  participant AM as ActionManager.cpp
  participant RM as ReplayMenu.cpp
  participant DX as dx8caps.cpp
  participant R2D as render2d.cpp

  Dev->>Repo: Apply PR #2285 patch
  Note over Repo: Comment-only edits

  Dev->>AM: Update TheSuperHackers date (06/01/2025 -> 06/01/2026)
  Dev->>RM: Update TheSuperHackers dates (07/02/2025 -> 07/02/2026)
  Dev->>DX: Update TheSuperHackers date (15/01/2025 -> 15/01/2026)
  Dev->>R2D: Update TheSuperHackers date (08/01/2025 -> 08/01/2026)

  Note over AM: No runtime behavior changes
  Note over RM: No runtime behavior changes
  Note over DX: No runtime behavior changes
  Note over R2D: No runtime behavior changes
Loading

Copy link

@Skyaero42 Skyaero42 left a comment

Choose a reason for hiding this comment

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

Looks good

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.

👀

@Caball009
Copy link
Author

Merged main branch to resolve conflicts.

@xezon xezon added Documentation Is documentation or complementary resource and removed Minor Severity: Minor < Major < Critical < Blocker labels Feb 10, 2026
@xezon xezon merged commit 77e4510 into TheSuperHackers:main Feb 10, 2026
24 checks passed
@Caball009 Caball009 deleted the fix_comment_dates_TSH branch February 10, 2026 18:44
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 Fix Is fixing something, but is not user facing ThisProject The issue was introduced by this project, or this task is specific to this project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants