correct `From<SmallVec<T, N, A>> for Box<[T]>` implementation
@HarshRajSinghania y travaille déjà.
Depuis le 4/10/2026.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 75/100
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
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
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de servo/rust-smallvec
-
add `arrayvec` feature for comparisons and conversionsPeut-être pris @GauravPawar101 l’a pris il y a 15 jours. Ouvertegood first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
servo/rust-smallvec#792 · 1 personne assignée ·
-
add `From<Box<[T]>, A> for SmallVec<T, N, A>` conversionPeut-être pris @alejandro-vaz l’a pris il y a 15 jours. Ouvertegood first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
servo/rust-smallvec#716 · 1 commentaire · 2 personnes assignées ·
-
Allocators are stable on nightly RustPeut-être pris @GauravPawar101 l’a pris il y a 36 jours. Ouvertegood first issue help wanted
Difficulté 1/5 Moins d'une heure Accessibilité débutants 70/100
servo/rust-smallvec#662 · 3 commentaires · 3 réactions · 1 personne assignée ·
-
fix workflow `if` conditions to correct repositoryPeut-être pris @GauravPawar101 l’a pris aujourd’hui. Ouvertegood first issue help wanted
servo/rust-smallvec#854 · 1 personne assignée ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 22/100
servo/rust-smallvec#848 · 2 commentaires ·
Toutes les issues de servo/rust-smallvec
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
aaif-goose/goose#13364 ·
Les mainteneurs répondent en général sous 1 jour
-
ai_p2
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
ClickHouse/ClickHouse#130006 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: PostgreSQL backend: Queue.remove() / Job.remove() never emit the QueueEvents "removed" eventPeut-être pris @1AbdullahButt l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
taskforcesh/bullmq#5085 ·
Les mainteneurs répondent en général sous 1 jour