AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Add an easier way to get both active & inactive views for a display
- Vorherrschende Sprache
- C++
- Sterne
- 2.1k
- Forks
- 503
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
In OCIO v1, the Config getViews(display) method returned all views. This was considered a bug (per issue #67) and a fix was attempted in closed PR #324 and then finally fixed in PR #875 (as part of OCIO v2).
As a result, in OCIO 2.x, getViews only returns the active views. This is usually what an application would want.
However in some situations it is useful to get all the views (active & inactive). Currently there is this API to do that:
```
int getNumViews(ViewType type, const char * display) const;
const char * getView(ViewType type, const char * display, int index) const;
```
but it requires iterating over both the shared and per-display views for a display. It would be nice to have the equivalent of getNumDisplaysAll and getDisplayAll for views.
Beitragsleitfaden
Rechercherichtung
Beginnen Sie mit dem Lesen von Config::getViews, getNumViews und getView. Vergleichen Sie anschließend deren Verhalten bei aktiven Views mit den entsprechenden getNumDisplaysAll- und getDisplayAll-APIs. Ermitteln Sie die vorgesehene kombinierte API für aktive und inaktive Views und überprüfen Sie, dass Aufrufer alle Views für ein Display abrufen können, ohne Shared- und Pro-Display-Views separat zu durchlaufen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100