Should Element store a constant pointer to the mesh?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 30/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- computer-graphics
Direzione di ricerca
Start by inspecting the Element and RangeSetBase classes and the existing constant-reference iteration behavior. Trace every place these classes use their mesh pointers to determine whether non-constant mesh access is required; done means documenting the remaining requirements and confirming whether both pointers can safely become pointers to constant meshes.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I have noticed that it is currently not possible to iterate over the elements of a constant reference - even if it is read only. However it seems to me that there is no reason for the Element and RangeSetBase class to store a nonconstant pointer. Note that this does NOT mean, that the pointer cannot change, we can still set it to nullptr or a different mesh - only the value that the pointer points to cannot, i.e. we would not be allowed to change element.getMesh()
I have implemented this for the Element class and it seems to work flawlessly. Is there any point where the elements require non-constant references?
- Lingua principale
- C++
- Stelle
- 1.3k
- Fork
- 183
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di nmwsharp/geometry-central
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
nmwsharp/geometry-central#256 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
nmwsharp/geometry-central#255 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
nmwsharp/geometry-central#253 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 57/100
nmwsharp/geometry-central#247 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
nmwsharp/geometry-central#246 ·
Tutte le issue di nmwsharp/geometry-central
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
Component: GLib
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
brave/brave-browser#59300 ·
-
Mute ydb/tests/functional/dstool/test_canonical_requests.py.Test.test_group_take_snapshot in main Apertaai_reviewed
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
ydb-platform/ydb#53974 · 3 commenti ·