AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()

Aperta
#2,255 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Modernise (C++17)
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.