AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
BuiltinTransform and Config Registry Get() return types
- Vorherrschende Sprache
- C++
- Sterne
- 2.1k
- Forks
- 503
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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?
Beitragsleitfaden
Rechercherichtung
Beginne mit dem Vergleich von BuiltinTransformRegistry::Get() und BuiltinConfigRegistry::Get(), einschließlich der Verwendung ihrer unterschiedlichen Rückgabetypen. Als erledigt gilt die Arbeit, wenn eine klare API-Entscheidung darüber getroffen wurde, ob die Rückgabetypen konsistent sein sollten, und die erwartete Änderung dokumentiert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- backend-api-design
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 30/100