Skip to content

refactor(GameEngine): Qualify WWVegas header includes#2872

Open
CryoTheRenegade wants to merge 1 commit into
TheSuperHackers:mainfrom
CryoTheRenegade:refactor/wwvegas-core-engine
Open

refactor(GameEngine): Qualify WWVegas header includes#2872
CryoTheRenegade wants to merge 1 commit into
TheSuperHackers:mainfrom
CryoTheRenegade:refactor/wwvegas-core-engine

Conversation

@CryoTheRenegade

Copy link
Copy Markdown

Relates to #2825

PR should be Squashed and Merged

This is the first in a series of PRs I am working on that migrate all of the header paths into the correct header paths.

Qualifies WWVegas header includes used by Core GameEngine, GameEngineDevice, debug, and profile sources.
This PR does not change CMake include-directory behavior, that will come later when all other headers are corrected.

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown

Greptile Summary

This PR qualifies WWVegas header includes across the engine and supporting libraries.

  • Updates GameEngine public and network includes.
  • Updates GameEngineDevice WW3D2 and WWMath includes.
  • Updates debug and profile WWLib includes.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
Core/GameEngine/Include/Common/OptionPreferences.h Qualifies the texture-filter header include.
Core/GameEngineDevice/Source/W3DDevice/GameClient/BaseHeightMap.cpp Qualifies WW3D2 and WWMath source includes.
Core/GameEngine/Source/GameNetwork/ConnectionManager.cpp Qualifies WWLib utility includes.
Core/Libraries/Source/debug/debug_debug.cpp Qualifies the WWCommon include.

Reviews (3): Last reviewed commit: "refactor(GameEngine): Qualify WWVegas he..." | Re-trigger Greptile

Comment thread Core/GameEngine/Include/Common/OptionPreferences.h Outdated
Comment thread Core/GameEngineDevice/Include/W3DDevice/GameClient/BaseHeightMap.h Outdated
@CryoTheRenegade CryoTheRenegade force-pushed the refactor/wwvegas-core-engine branch from 80a4eec to 5ca91b8 Compare July 13, 2026 22:21
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.

1 participant