AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

MathUtils.cpp/h: Unused functions / duplicated functionality

Offen
#2,259 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.