Skip to content

Move PhysicServer enums, structs and sub-objects to dedicated files#120983

Open
smix8 wants to merge 1 commit into
godotengine:masterfrom
smix8:physics_mambo
Open

Move PhysicServer enums, structs and sub-objects to dedicated files#120983
smix8 wants to merge 1 commit into
godotengine:masterfrom
smix8:physics_mambo

Conversation

@smix8

@smix8 smix8 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Moves PhysicServer enums, structs and sub-objects to dedicated files.

Someone had to do it and slay that monolith spaghetti code.
Everything in the engine seem to want to include the full physics server.

2D

The old enums can now all be found under servers/physics_2d/physics_server_2d_enums.h
The old structs can now all be found under servers/physics_2d/physics_server_2d_types.h
The old constants can now all be found under servers/physics_2d/physics_server_2d_constants.h
The old direct body state and space state objects can be all found in folder servers/physics_2d/direct_states
The old parameter and results can now all be found under servers/physics_2d/queries

3D

The old enums can now all be found under servers/physics_3d/physics_server_3d_enums.h
The old structs can now all be found under servers/physics_3d/physics_server_3d_types.h
The old constants can now all be found under servers/physics_3d/physics_server_3d_constants.h
The old direct body state and space state objects can be all found in folder servers/physics_3d/direct_states
The old parameter and results can now all be found under servers/physics_3d/queries

There are many more include fixes possible but I did run out of energy and just want that abomination-thing to be over before I lose my sanity on the next forced rebase.

No AI was used because I have a glimpse of self-dignity ... can't speak for my sanity.
reeee

@smix8 smix8 requested review from a team as code owners July 5, 2026 21:06
Moves PhysicServer enums, structs and sub-objects to dedicated files.

@Repiteo Repiteo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pain is mandatory.

@Repiteo Repiteo added this to the 4.8 milestone Jul 6, 2026
@Repiteo Repiteo removed request for a team July 6, 2026 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants