Reduce compile time by removing geometry_3d.h include from rendering_server.h

[Ivorforce]

• Helps address [TRACKER] Compile time issues caused by excessive includes #111218
• Supersedes part of Core: Clean up Geometry2D and Geometry3D #111602

The change reduces compile time, especially if geometry_3d.h or delauney_3d.h are edited.

I needed to change Geometry3D from class to namespace to make it happen, otherwise it would be impossible to forward-declare Geometry3D::MeshData. There are no further changes to this file (most of the changelog is whitespace).

This makes sense anyhow since Geometry3D does not need to be instantiated.

[clayjohn]

Makes sense to me. It looks like there are no class members or private functions or really any reason not to make Geometry3D a namespace.

The only thing that isn't clear to me on the face of the code is how this affects GDScript/GDExtension bindings/doc generation. Is there any reason to think that bindings may be impacted by this change?

[Ivorforce]

The only thing that isn't clear to me on the face of the code is how this affects GDScript/GDExtension bindings/doc generation. Is there any reason to think that bindings may be impacted by this change?

Nope! Geometry3D is bound via CoreBind::Geometry3D, which is a separate class we don't use internally.
That means Geometry3D will continue to be a class in GDScript / godot-cpp / GDExtension.

[clayjohn]

Sounds great! Thanks for the quick response

[Repiteo]

Thanks!