AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
- Lingua principale
- C++
- Stelle
- 2.1k
- Fork
- 503
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
As we are now C++17 or greater we should probably investigate replacing calls to Clamp() with std::clamp.
Care may need with both performance and behaviour of non-normals etc to ensure the behaviour remains unchanged - for example ordering of arguments when one of them is a Nan, handling of +0.0 vs -0.0 etc
https://en.cppreference.com/w/cpp/algorithm/clamp.html
At first glance our implementation appears to be directly substitutable.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia individuando l’implementazione esistente di Clamp() e tutti i suoi punti di chiamata, quindi confrontali con il riferimento C++17 std::clamp collegato. Il lavoro è completato quando si determina se la sostituzione preserva le prestazioni e il comportamento per l’ordine degli argomenti NaN e per +0.0 rispetto a -0.0, identificando tutti gli utilizzi interessati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- computer-graphics, performance
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 38/100