AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

MathUtils.h: header functions should be replaced with definitions from half.h

Aperta
#2,254 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
help wanted
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.