amethyst / amethyst/sheep

Named format uses `Vec<String>` instead of `&[String]`

Aperta
#13 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement help wanted
Lingua principale
Rust
Stelle
93
Fork
12
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Currently, `AmethystNamedFormat` takes extra options in the form `Vec`. This is suboptimal since it requires moving the whole container and cloning it if it needs to be reused. Instead it could be `&[String]`, or even `&[S]` with `AmethystNamedFormat where S: Deref`.

I'm happy to write up a PR for this if people think it's a good idea.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.