AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
MathUtils.cpp/h: Unused functions / duplicated functionality
- Vorherrschende Sprache
- C++
- Sterne
- 2.1k
- Forks
- 505
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Rechercherichtung
Beginne mit dem Lesen der Deklarationen und Implementierungen in MathUtils.cpp und MathUtils.h und vergleiche anschließend die 4x4-Matrixoperationen mit der Inversen in MatrixOpData.cpp. Überprüfe den Testcode, der std::isnan aufruft. Für den Abschluss ist eine Entscheidung der Maintainer über die Entfernung oder weitere Verwendung erforderlich, wobei der betroffene Code und die Tests konsistent aktualisiert werden müssen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100