AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

BuiltinTransform and Config Registry Get() return types

Abierto
#2,156 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
C++
Estrellas
2.1k
Forks
503
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

It was noticed that `BuiltinTransformRegistry::Get()` and `BuiltinConfigRegistry::Get()` are both similar looking methods but the former return a shared pointer while the latter a reference. Should we try to have both of them returning a shared pointer for example to make the API more consistent?

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza comparando BuiltinTransformRegistry::Get() con BuiltinConfigRegistry::Get(), incluido cómo se utilizan sus tipos de retorno diferentes. Se considera terminado cuando se haya tomado una decisión clara de API sobre si los tipos de retorno deben ser coherentes y se haya documentado el cambio esperado.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
backend-api-design
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.