AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Const correctness of PackedImageDesc
- Langage dominant
- C++
- Étoiles
- 2.1k
- Forks
- 505
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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?
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par suivre PackedImageDesc dans CPUProcessor::apply et examiner les fonctions getData mentionnées dans l’issue. Déterminez le comportement visé en matière de const-correctness et si un ConstPackedImageDesc distinct ou une portée plus restreinte limitée à apply est approprié ; le travail est considéré comme terminé lorsque l’orientation de l’API est convenue et spécifiée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- backend-api-design
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100