AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
MathUtils.h: header functions should be replaced with definitions from half.h
- Lingua principale
- C++
- Stelle
- 2.1k
- Fork
- 503
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
In MathUtils.h there are some helper functions that provide hard coded numbers for some of the half data format numeric limits:
GetHalfMax, GetHalfMin, GetHalfNormMin
These should be replaced with definitions pulled from half.h
As they also appear to not be either unused or only used by MathUtils.cpp I would propose they be first removed/migrated into that translation unit and then replaced.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia esaminando gli helper GetHalfMax, GetHalfMin e GetHalfNormMin in MathUtils.h, i loro utilizzi in MathUtils.cpp e le definizioni corrispondenti in half.h. Verifica che gli helper possano essere rimossi o spostati in MathUtils.cpp, quindi controlla che i limiti numerici provengano da half.h e che i test esistenti continuino a passare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- backend
- Tipo di issue
- Refactoring
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100