rust-lang/rust-clippy

False positive for boxed local with `wasm_bindgen`

Open

#4 351 ouverte le 7 août 2019

Voir sur GitHub
 (14 commentaires) (0 réactions) (0 assignés)Rust (1 391 forks)batch import
C-bugI-false-positivegood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Inspectez le lint Clippy qui se déclenche pour les boxed locals et comprenez l'interaction avec la génération de code de wasm bindgen. Déterminez pourquoi les boxed slices devraient être autorisées comme paramètres locaux dans ce contexte.
Stack technique
rust
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65