AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Const correctness of PackedImageDesc

Aperta
#2,114 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

In our use case, we are passing const void pointers to build a source `PackedImageDesc` for use in `CPUProcessor::apply`. However, `PackedImageDesc` requires a non-const pointer. I assume this is so that users with non-const data can call `getData` and modify the result. This makes it dangerous to const_cast our pointer as a caller could then end up inadvertently modifying a const buffer through those calls.

Perhaps there should be a `ConstPackedImageDesc`? Or is the design to keep the scope of `PackedImageDesc` tightly around the `apply` call so that it's unlikely that users can access the `getData` functions?

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia tracciando PackedImageDesc attraverso CPUProcessor::apply e esaminando le funzioni getData menzionate nell’issue. Determina il comportamento previsto in termini di const-correctness e se sia appropriato un ConstPackedImageDesc separato o un ambito più ristretto limitato a apply; il lavoro è completato quando la direzione dell’API è concordata e specificata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
backend-api-design
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.