AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
- Langage dominant
- C++
- Étoiles
- 2.1k
- Forks
- 503
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par localiser l’implémentation existante de Clamp() et tous ses sites d’appel, puis comparez-les à la référence C++17 std::clamp liée. Le travail est terminé lorsqu’il est déterminé si le remplacement préserve les performances et le comportement pour l’ordre des arguments NaN ainsi que pour +0.0 par rapport à -0.0, avec toutes les utilisations affectées identifiées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- computer-graphics, performance
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 38/100