Skip to content

refactor(WW3D2): Qualify Core header includes#2873

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

refactor(WW3D2): Qualify Core header includes#2873
CryoTheRenegade wants to merge 1 commit into
TheSuperHackers:mainfrom
CryoTheRenegade:refactor/wwvegas-ww3d2-core

Conversation

@CryoTheRenegade

@CryoTheRenegade CryoTheRenegade commented Jul 13, 2026

Copy link
Copy Markdown

PR should be Squashed and Merged

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

Qualifies cross-component header includes throughout Core WW3D2 with their owning WWVegas component paths.

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown

Greptile Summary

This PR qualifies WWVegas component header includes across Core WW3D2. The main changes are:

  • Adds component prefixes such as WWLib/, WWMath/, and WW3D2/.
  • Updates shared dependency includes in WW3D2 headers and source files.
  • Leaves runtime code and CMake include-directory behavior unchanged.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
Core/Libraries/Source/WWVegas/WW3D2/aabtree.h Qualifies WWLib, WWMath, WWDebug, and WW3D2 header includes.
Core/Libraries/Source/WWVegas/WW3D2/animatedsoundmgr.cpp Qualifies WWLib, WWSaveLoad, WWAudio, and WWDebug dependencies.
Core/Libraries/Source/WWVegas/WW3D2/dx8wrapper.cpp Qualifies shared WWVegas component includes without changing implementation logic.

Reviews (1): Last reviewed commit: "refactor(WW3D2): Qualify headers in aabt..." | Re-trigger Greptile

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