[ML] Migrate CPersistUtils to use std::string_view
Offen
>enhancement
- Vorherrschende Sprache
- C++
- Sterne
- 157
- Forks
- 67
- Ø Merge
- 17 Std. 52 Min.
- Gemergte PRs (30 T.)
- 20
Beschreibung
Now that main is on C++17 we can migrate `CPersistUtils` to use `std::string_view`. This would allow us to eliminate all string copies where we extract parts of a `std::string`.
Beitragsleitfaden
Rechercherichtung
Beginne damit, die Deklaration von CPersistUtils und alle Aufrufstellen der String-Extraktion zu finden. Verfolge, wie diese Teile von std::string verwendet werden, und migriere dann die relevanten Schnittstellen zu std::string_view, wobei das Verhalten erhalten bleibt. Baue das Projekt und führe die vorhandene Testsuite aus, um zu bestätigen, dass String-Kopien ohne Regressionen entfernt wurden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100