AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Const correctness of PackedImageDesc
- 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