Improve error message for missing #[sp(var_size)] : attempt to compute <...> wich would overflow
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia tracciando il percorso di controllo delle dimensioni di simple_parse e il suo utilizzo del crate static_assertions, quindi esamina Rust issue #51999 per le limitazioni diagnostiche menzionate. Il lavoro è completo quando l'output di overflow confuso viene sostituito da un errore che identifica il campo della struct o dell'enum e raccomanda #[sp(var_size)] quando possibile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Currently, simple_parse ensures that it does not mistake static/dynamic sizes that it does not know about through the static_assertions crate. Unfortunately when a const assert fails, you get an ugly error message that is confusing at best :
attempt to compute `0_usize - 1_usize`, which would overflow
This error can be fixed by annotating your dynamic fields with #[sp(var_size)]
It would be nice to have an error that says :
- Where the annotation is needed (which field of which struct/enum)
- What needs to be added (the sp attribute above)
This might not be currently possible because of https://github.com/rust-lang/rust/issues/51999
- Lingua principale
- Rust
- Stelle
- 13
- Fork
- 1
- 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 elast0ny/simple_parse
-
Write tests Apertaenhancement good first issue
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
elast0ny/simple_parse#2 · 1 commento · 1 reazione ·
Tutte le issue di elast0ny/simple_parse
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
state:needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
zed-industries/zed#64680 · 2 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
RustPython/RustPython#8802 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
TheLarkInn/aipm#2390 ·