correct `From<SmallVec<T, N, A>> for Box<[T]>` implementation

Fermée Adaptée aux débutants
  • Quelque chose ne va pas ?
#714 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@HarshRajSinghania y travaille déjà.

Depuis le 4/10/2026.

  • #732 par @HarshRajSinghania — ouverte
  • #734 par @bolshoytoster — fusionnée

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
75/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
rust
Domaine
backend

Piste de recherche

Commencez dans conversions.rs et comparez la conversion existante avec d’autres implémentations prenant en charge les allocateurs. Mettez à jour la conversion SmallVec-to-box afin que l’allocateur soit conservé sans Vec intermédiaire, puis vérifiez que la Box obtenue utilise l’allocateur d’origine et ajoutez ou ajustez une couverture ciblée pour cette conversion.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

good first issue help wanted r-feature

the current implementation on conversions.rs:

impl<T, const N: usize, A: Allocator> From<SmallVec<T, N, A>> for Box<[T]> {
    fn from(this: SmallVec<T, N, A>) -> Self {
        Vec::from(this).into_boxed_slice()
    }
}

first converts it to a Vec and changes the allocator before converting it into a box in Global

the correct implementation should preserve the allocator and not make use of an intermediary Vec:

impl<T, const N: usize, A: Allocator> From<SmallVec<T, N, A>> for Box<[T], A>
Langage dominant
Rust
Étoiles
1.8k
Forks
242
Merge moyen
1 j 5 h
PR mergées (30 j)
52

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de servo/rust-smallvec

Toutes les issues de servo/rust-smallvec

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.