Consider using a pointer offset instead of a slice and integer position
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- performance
Línea de trabajo
Comienza en src/tokenizer.rs alrededor de la línea 372, donde next_byte_unchecked es utilizado por código como skip_whitespace. Compara el ensamblado generado para el enfoque actual de slice y offset con el enfoque de offset mediante puntero propuesto y, a continuación, verifica que el comportamiento del tokenizer no haya cambiado y que el código generado tenga menos indirecciones.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The generated assembly for code using next_byte_unchecked like skip_whitespace has a suprising number of indirections in order to actually get the byte value. This is caused by the need to get the slice, then get the byte at the desired offset from it at https://github.com/servo/rust-cssparser/blob/682087fca5ba5f2f05a09bba72c62dac6b3d778d/src/tokenizer.rs#L372. If we use store a pointer and increase its instead of the offset, it should cause more efficient code to be generated.
- Lenguaje dominante
- Rust
- Estrellas
- 869
- Forks
- 152
- Merge medio
- 15 h 8 min
- PR fusionados (30 d)
- 12
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 servo/rust-cssparser
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
servo/rust-cssparser#440 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
servo/rust-cssparser#439 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
servo/rust-cssparser#434 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
servo/rust-cssparser#438 · 3 comentarios · 1 reacción ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 28/100
servo/rust-cssparser#436 · 1 comentario ·
Todos los issues de servo/rust-cssparser
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
state:needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
zed-industries/zed#64680 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
RustPython/RustPython#8802 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
TheLarkInn/aipm#2390 ·