AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

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

Abierto
#2,255 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Modernise (C++17)
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.