AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

BuiltinTransform and Config Registry Get() return types

Aperta
#2,156 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.