elastic / elastic/ml-cpp

[ML] Migrate CPersistUtils to use std::string_view

Open
#2,340 0 comments 0 reactions 0 assignees View on GitHub
>enhancement
Dominant language
C++
Stars
157
Forks
67
Avg merge
12h 48m
Merged PRs (30d)
16

Description

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`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.