AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

BuiltinTransform and Config Registry Get() return types

Open
#2,156 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2.1k
Forks
503
PR merge metrics
No merged PRs in 30d

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?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.