MathUtils.cpp/h: Unused functions / duplicated functionality

In MathUtils.cpp there are a number of functions that are apparently not used by the rest of the code:

GetSafeScalarInverse
VecContainsZero
VecContainsOne
GetM44Inverse
GetM44M44Product
GetM44V4Product
GetV4Sum
GetMxbCombine
GetMxbInverse

Should they be used int he code where we are not? or should the code be removed? I'm more concerned with the 4x4 matrix bits as they don't look the most robust numerically, and we have code in MatrixOpData.cpp for a better inverse.

I also note we have a IsNan function that is a simple wrapper for std::isnan do we need it? If we want to keep it we should update the test code the calls std::isnan to use the wrapper.

Inizia leggendo le dichiarazioni e le implementazioni in MathUtils.cpp e MathUtils.h, quindi confronta le operazioni sulle matrici 4x4 con l’inversa in MatrixOpData.cpp. Controlla il codice dei test che chiama std::isnan. Per considerarlo completato è necessaria una decisione di un maintainer sulla rimozione o sul suo utilizzo continuato, aggiornando in modo coerente il codice e i test interessati.

Scritto dal modello di indicizzazione a partire dal testo della issue.