servo/servo

Don't use `to_string` for owned `DOMString` if it is not gonna be used again. Use `String::from`

Open

#44.999 aberto em 19 de mai. de 2026

Ver no GitHub
 (3 comments) (0 reactions) (1 assignee)Rust (36.724 stars) (3.630 forks)batch import
C-assignedE-less-complexI-cleanupI-perf-bloatI-perf-slowgood first issue

Description

String::from

https://github.com/servo/servo/blob/efac0e90cc0f43fb262887238530703499e246b5/components/script_bindings/domstring.rs#L914-L918

consumes the DOMString and is straightforward.

to_string

https://github.com/servo/servo/blob/efac0e90cc0f43fb262887238530703499e246b5/components/script_bindings/domstring.rs#L808-L812

goes through Formatter, which is less efficient.

Guia do colaborador

Pilha de tecnologia
rust
Domain
performance
Tipo Issue
performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
basics of Rust ownership
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90
Direção de pesquisa
Replace the two `to string()` calls on lines 808-812 and 914-918 of `components/script bindings/domstring.rs` with `String::from()`. The former goes through Formatter and is less efficient, while the latter consumes the DOMString directly. Ensure the variable is not used after the conversion.