Improve error message for missing #[sp(var_size)] : attempt to compute <...> wich would overflow
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Empieza siguiendo la ruta de comprobación de tamaño de simple_parse y su uso del crate static_assertions; después, revisa Rust issue #51999 para conocer las limitaciones de diagnóstico mencionadas. Se considera terminado cuando se reemplace la salida de overflow confusa por un error que identifique el campo de la struct o enum y recomiende #[sp(var_size)] cuando sea viable.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Rust
- Estrellas
- 13
- Forks
- 1
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de elast0ny/simple_parse
-
Write tests Abiertoenhancement good first issue
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
elast0ny/simple_parse#2 · 1 comentario · 1 reacción ·
Todos los issues de elast0ny/simple_parse
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
bug good first issue package: quic
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
`dora trace view` sends a non-canonical full UUID as-is, so a valid trace ID shows "No spans found" Abiertocli coordinator rust
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
area: tasks enhancement good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Jason-jo17/Polybench#15 · 1 comentario ·