altmp/altv-js-module-v2

Switch to `std::string_view` instead of `const std::string&`

Closed this issue · 1 comments

We are using C++20 anyway so why not make use of some modern C++ features

Well, after more evaluation I have come to the conclusion that this would make writing code a lot more cumbersome (e.g. you can't concatenate string_view's or implicitly convert them to std::string) so yeah its not worth it