amethyst / amethyst/sheep

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

Offen
#13 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement help wanted
Vorherrschende Sprache
Rust
Sterne
93
Forks
12
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.