AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
- Lenguaje dominante
- C++
- Estrellas
- 2.1k
- Forks
- 503
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
Línea de trabajo
Empieza localizando la implementación existente de Clamp() y todos sus puntos de llamada, y compáralos después con la referencia enlazada de C++17 std::clamp. Se considerará terminado cuando se determine si la sustitución conserva el rendimiento y el comportamiento para el orden de los argumentos NaN y para +0.0 frente a -0.0, con todos los usos afectados identificados.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- computer-graphics, performance
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100