AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
BuiltinTransform and Config Registry Get() return types
- Lingua principale
- C++
- Stelle
- 2.1k
- Fork
- 503
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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?
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia confrontando BuiltinTransformRegistry::Get() con BuiltinConfigRegistry::Get(), incluso il modo in cui vengono utilizzati i rispettivi tipi di ritorno diversi. Il lavoro è completato quando è stata presa una decisione chiara sull’API riguardo alla necessità che i tipi di ritorno siano coerenti e la modifica prevista è stata documentata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- backend-api-design
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 30/100