rust-lang/rust-clippy

False positive for boxed local with `wasm_bindgen`

Open

#4351 aperta il 7 ago 2019

Vedi su GitHub
 (14 commenti) (0 reazioni) (0 assegnatari)Rust (1391 fork)batch import
C-bugI-false-positivegood first issue

Metriche repository

Star
 (10.406 star)
Metriche merge PR
 (Merge medio 19g 22h) (113 PR mergiate in 30 g)

Descrizione

wasm_bindgen is a little bit different than the issue that's been resolved with traits. In particular, wasm_bindgen just generates code around the non-trait implemented function, and the only way to use native JS arrays as parameters is to use Box<[...]>.

In general, I think maybe boxed slices should be considered ok for local parameters, as they're conceptually very similar to vectors.

Guida contributor

Direzione di ricerca
Ispeziona il lint di clippy che si attiva per i boxed locals e comprendi l'interazione con la generazione di codice di wasm bindgen. Determina perché i boxed slices dovrebbero essere consentiti come parametri locali in questo contesto.
Tech stack
rust
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65