Improve type safety of internal classes
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
Direzione di ricerca
Inizia facendo l’inventario delle classi interne e dei cast dalla classe base alla classe derivata interessati dall’issue, quindi traccia il punto in cui questi oggetti entrano nel motore di strumentazione. Confronta gli attuali pattern di ereditarietà e delle interfacce con l’approccio proposto dell’ereditarietà virtuale e del QI nella classe derivata; il lavoro è completato quando i cast non sicuri rilevanti sono stati risolti e i percorsi interessati rimangono compatibili.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
There are a number of places where we cast from a base class to a derived class in our internal code, which could cause problems if a caller passes us an instance of an interface that we did not create internally. I suggest we:
-
Update our code to use virtual inheritance (instead of multiple inheritance). This will help us catch locations where we cast from a base class.
-
QI for the derived class. This will give us type safety.
- Lingua principale
- C++
- Stelle
- 104
- Fork
- 53
- Merge medio
- 3h 34m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 microsoft/CLRInstrumentationEngine
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
microsoft/CLRInstrumentationEngine#169 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
microsoft/CLRInstrumentationEngine#526 · 1 reazione ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
Tutte le issue di microsoft/CLRInstrumentationEngine
Issue simili
-
Status: Awaiting triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
espressif/arduino-esp32#12984 ·
I maintainer di solito rispondono entro 1 giorno
-
torch_ops/logprob.cu does not compile with the serving container's nvcc (13.3.73); check_torch_ops.py cannot run as shippedForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 72/100
ashhart/TensorFold#535 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno