Hi,
I ran Cppcheck for static code analysis on the project and noticed that we have these 4 parameters that are non-reference const objects even though we could use const reference types which might be more performant in the case of std::string. And looking at some other functions, it seems like const std::string& is preferred over const std::string in the rest of the project.
Hi, I ran Cppcheck for static code analysis on the project and noticed that we have these 4 parameters that are non-reference const objects even though we could use const reference types which might be more performant in the case of
std::string
. And looking at some other functions, it seems likeconst std::string&
is preferred overconst std::string
in the rest of the project.