Externals/gtest/src/gtest.cc:3706] (performance) The conversion from const char* as returned by c_str() to std:
tring creates an unnecessary string copy. Solve that by directly passing the string. [stlcstrParam]
Externals/glslang/glslang/MachineIndependent/propagateNoContraction.cpp:534] (performance) Either inefficient or wrong usage of string::find(). string:tarts_with() will be faster if string::find's result is compared with 0, because it will not scan the whole string. If your intention is to check that there are no findings in the string, you should compare with std:tring::npos. [stlIfStrFind]
Externals/glslang/glslang/MachineIndependent/propagateNoContraction.cpp:539] (performance) Either inefficient or wrong usage of string::find(). string:tarts_with() will be faster if string::find's result is compared with 0, because it will not scan the whole string. If your intention is to check that there are no findings in the string, you should compare with std:tring::npos. [stlIfStrFind]
Externals/glslang/glslang/MachineIndependent/LiveTraverser.h:122] (performance) Searching before insertion is not necessary. [stlFindInsert]
Externals/glslang/SPIRV/hex_float.h:125] (performance) When an object of a class is created, the constructors of all member variables are called consecutively in the order the variables are declared, even if you don't explicitly write them to the initialization list. You could avoid assigning 'data_' a value by passing the value to the constructor in the initialization list. [useInitializationList]
Externals/glslang/SPIRV/SpvBuilder.h:514] (performance) Parameter 'newChain' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++. [passedByValue]
Externals/glslang/StandAlone/spirv-remap.cpp:61] (performance) Parameter 'filename' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++. [passedByValue]
tring creates an unnecessary string copy. Solve that by directly passing the string. [stlcstrParam]Externals/glslang/glslang/MachineIndependent/propagateNoContraction.cpp:534] (performance) Either inefficient or wrong usage of string::find(). string:
tarts_with() will be faster if string::find's result is compared with 0, because it will not scan the whole string. If your intention is to check that there are no findings in the string, you should compare with std:tring::npos. [stlIfStrFind]Externals/glslang/glslang/MachineIndependent/propagateNoContraction.cpp:539] (performance) Either inefficient or wrong usage of string::find(). string:
tarts_with() will be faster if string::find's result is compared with 0, because it will not scan the whole string. If your intention is to check that there are no findings in the string, you should compare with std:tring::npos. [stlIfStrFind]Externals/glslang/glslang/MachineIndependent/LiveTraverser.h:122] (performance) Searching before insertion is not necessary. [stlFindInsert]
Externals/glslang/SPIRV/hex_float.h:125] (performance) When an object of a class is created, the constructors of all member variables are called consecutively in the order the variables are declared, even if you don't explicitly write them to the initialization list. You could avoid assigning 'data_' a value by passing the value to the constructor in the initialization list. [useInitializationList]
Externals/glslang/SPIRV/SpvBuilder.h:514] (performance) Parameter 'newChain' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++. [passedByValue]
Externals/glslang/StandAlone/spirv-remap.cpp:61] (performance) Parameter 'filename' is passed by value. It could be passed as a const reference which is usually faster and recommended in C++. [passedByValue]
