AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
BuiltinTransform and Config Registry Get() return types
- Langage dominant
- C++
- Étoiles
- 2.1k
- Forks
- 505
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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?
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par comparer BuiltinTransformRegistry::Get() et BuiltinConfigRegistry::Get(), notamment la manière dont leurs types de retour différents sont utilisés. Le travail est considéré comme terminé lorsqu’une décision claire concernant l’API a été prise quant à la cohérence nécessaire des types de retour et que la modification attendue est documentée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- backend-api-design
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 30/100