clang-tidy: use const references

Found with performance-unnecessary-value-param

Signed-off-by: Rosen Penev <rosenp@gmail.com>
This commit is contained in:
Rosen Penev 2023-12-21 18:16:45 -08:00 committed by Urs Fleisch
parent 1635d4d563
commit b07db7510f
2 changed files with 3 additions and 3 deletions

View File

@ -65,7 +65,7 @@ String Utils::pictureTypeToString(int type)
return "";
}
int Utils::pictureTypeFromString(String str)
int Utils::pictureTypeFromString(const String& str)
{
for(int i = 0; i < static_cast<int>(std::size(typeStrs)); ++i) {
if(str == typeStrs[i]) {

View File

@ -93,7 +93,7 @@ enum name { \
static TagLib::String typeToString(name type) { \
return TagLib::Utils::pictureTypeToString(type); \
} \
static name typeFromString(TagLib::String str) { \
static name typeFromString(const TagLib::String &str) { \
return static_cast<name>( \
TagLib::Utils::pictureTypeFromString(str)); \
}
@ -112,7 +112,7 @@ namespace TagLib {
/*!
* Get picture type from string representation.
*/
int TAGLIB_EXPORT pictureTypeFromString(String str);
int TAGLIB_EXPORT pictureTypeFromString(const String& str);
} // namespace Utils
} // namespace TagLib