AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

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

Ouverte
#2,255 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Modernise (C++17)
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.