Skip to content

Remove "z" from "vertex_t"

Nev3r requested to merge remove-vertex-z-mk2 into next

This removes vertex_t's z variable, as it is unnecessary. The only occurrence across the entire codebase for this variable was a hack used by the rope hang snapping behavior; this now uses vector3_t for the necessary calculations. P_ClosestPointOnLine3D() was also adapted to make use of vector3_t. (A similar function, FV3_ClosestPointOnLine(), is meant to have the same functionality, but the math there fails, possibly due to fixed point overflows, and therefore P_ClosestPointOnLine3D() was kept and adapted for this purpose.)

Merge request reports