diff options
author | Jonas Karlsson <jonas.karlsson@qt.io> | 2020-05-27 12:34:29 +0200 |
---|---|---|
committer | Jonas Karlsson <jonas.karlsson@qt.io> | 2020-06-03 18:56:21 +0200 |
commit | 1b33ee95e5c6e5e27f732fd273920861fdae486a (patch) | |
tree | f8e77cab65d4d32df4f65ee155e01b75e5993598 /tests/auto/gui/rhi/qshader/tst_qshader.cpp | |
parent | 1f88ccc8ad64096209cdf7874af129f4220b00ba (diff) |
Use QByteArray instead of QString
Since the variable names in QShaderDescription are later compared to
QByteArrays we can gain some performance from not having to convert them
to QByteArrays later.
Task-Id: QTBUG-83706
Change-Id: Iaf80d0966f45cbb09e7c1000b7854bc488e57bb3
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Diffstat (limited to 'tests/auto/gui/rhi/qshader/tst_qshader.cpp')
-rw-r--r-- | tests/auto/gui/rhi/qshader/tst_qshader.cpp | 38 |
1 files changed, 19 insertions, 19 deletions
diff --git a/tests/auto/gui/rhi/qshader/tst_qshader.cpp b/tests/auto/gui/rhi/qshader/tst_qshader.cpp index 22173d6d1a..d12efa90dc 100644 --- a/tests/auto/gui/rhi/qshader/tst_qshader.cpp +++ b/tests/auto/gui/rhi/qshader/tst_qshader.cpp @@ -91,11 +91,11 @@ void tst_QShader::simpleCompileCheckResults() for (const QShaderDescription::InOutVariable &v : desc.inputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("position")); + QCOMPARE(v.name, QByteArrayLiteral("position")); QCOMPARE(v.type, QShaderDescription::Vec4); break; case 1: - QCOMPARE(v.name, QLatin1String("color")); + QCOMPARE(v.name, QByteArrayLiteral("color")); QCOMPARE(v.type, QShaderDescription::Vec3); break; default: @@ -107,7 +107,7 @@ void tst_QShader::simpleCompileCheckResults() for (const QShaderDescription::InOutVariable &v : desc.outputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("v_color")); + QCOMPARE(v.name, QByteArrayLiteral("v_color")); QCOMPARE(v.type, QShaderDescription::Vec3); break; default: @@ -117,8 +117,8 @@ void tst_QShader::simpleCompileCheckResults() } QCOMPARE(desc.uniformBlocks().count(), 1); const QShaderDescription::UniformBlock blk = desc.uniformBlocks().first(); - QCOMPARE(blk.blockName, QLatin1String("buf")); - QCOMPARE(blk.structName, QLatin1String("ubuf")); + QCOMPARE(blk.blockName, QByteArrayLiteral("buf")); + QCOMPARE(blk.structName, QByteArrayLiteral("ubuf")); QCOMPARE(blk.size, 68); QCOMPARE(blk.binding, 0); QCOMPARE(blk.descriptorSet, 0); @@ -129,14 +129,14 @@ void tst_QShader::simpleCompileCheckResults() case 0: QCOMPARE(v.offset, 0); QCOMPARE(v.size, 64); - QCOMPARE(v.name, QLatin1String("mvp")); + QCOMPARE(v.name, QByteArrayLiteral("mvp")); QCOMPARE(v.type, QShaderDescription::Mat4); QCOMPARE(v.matrixStride, 16); break; case 1: QCOMPARE(v.offset, 64); QCOMPARE(v.size, 4); - QCOMPARE(v.name, QLatin1String("opacity")); + QCOMPARE(v.name, QByteArrayLiteral("opacity")); QCOMPARE(v.type, QShaderDescription::Float); break; default: @@ -314,7 +314,7 @@ void tst_QShader::loadV3() for (const QShaderDescription::InOutVariable &v : desc.inputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("qt_TexCoord")); + QCOMPARE(v.name, QByteArrayLiteral("qt_TexCoord")); QCOMPARE(v.type, QShaderDescription::Vec2); break; default: @@ -326,7 +326,7 @@ void tst_QShader::loadV3() for (const QShaderDescription::InOutVariable &v : desc.outputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("fragColor")); + QCOMPARE(v.name, QByteArrayLiteral("fragColor")); QCOMPARE(v.type, QShaderDescription::Vec4); break; default: @@ -336,8 +336,8 @@ void tst_QShader::loadV3() } QCOMPARE(desc.uniformBlocks().count(), 1); const QShaderDescription::UniformBlock blk = desc.uniformBlocks().first(); - QCOMPARE(blk.blockName, QLatin1String("buf")); - QCOMPARE(blk.structName, QLatin1String("ubuf")); + QCOMPARE(blk.blockName, QByteArrayLiteral("buf")); + QCOMPARE(blk.structName, QByteArrayLiteral("ubuf")); QCOMPARE(blk.size, 68); QCOMPARE(blk.binding, 0); QCOMPARE(blk.descriptorSet, 0); @@ -348,14 +348,14 @@ void tst_QShader::loadV3() case 0: QCOMPARE(v.offset, 0); QCOMPARE(v.size, 64); - QCOMPARE(v.name, QLatin1String("qt_Matrix")); + QCOMPARE(v.name, QByteArrayLiteral("qt_Matrix")); QCOMPARE(v.type, QShaderDescription::Mat4); QCOMPARE(v.matrixStride, 16); break; case 1: QCOMPARE(v.offset, 64); QCOMPARE(v.size, 4); - QCOMPARE(v.name, QLatin1String("opacity")); + QCOMPARE(v.name, QByteArrayLiteral("opacity")); QCOMPARE(v.type, QShaderDescription::Float); break; default: @@ -477,7 +477,7 @@ void tst_QShader::loadV4() for (const QShaderDescription::InOutVariable &v : desc.inputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("qt_TexCoord")); + QCOMPARE(v.name, QByteArrayLiteral("qt_TexCoord")); QCOMPARE(v.type, QShaderDescription::Vec2); break; default: @@ -489,7 +489,7 @@ void tst_QShader::loadV4() for (const QShaderDescription::InOutVariable &v : desc.outputVariables()) { switch (v.location) { case 0: - QCOMPARE(v.name, QLatin1String("fragColor")); + QCOMPARE(v.name, QByteArrayLiteral("fragColor")); QCOMPARE(v.type, QShaderDescription::Vec4); break; default: @@ -499,8 +499,8 @@ void tst_QShader::loadV4() } QCOMPARE(desc.uniformBlocks().count(), 1); const QShaderDescription::UniformBlock blk = desc.uniformBlocks().first(); - QCOMPARE(blk.blockName, QLatin1String("buf")); - QCOMPARE(blk.structName, QLatin1String("ubuf")); + QCOMPARE(blk.blockName, QByteArrayLiteral("buf")); + QCOMPARE(blk.structName, QByteArrayLiteral("ubuf")); QCOMPARE(blk.size, 68); QCOMPARE(blk.binding, 0); QCOMPARE(blk.descriptorSet, 0); @@ -511,14 +511,14 @@ void tst_QShader::loadV4() case 0: QCOMPARE(v.offset, 0); QCOMPARE(v.size, 64); - QCOMPARE(v.name, QLatin1String("qt_Matrix")); + QCOMPARE(v.name, QByteArrayLiteral("qt_Matrix")); QCOMPARE(v.type, QShaderDescription::Mat4); QCOMPARE(v.matrixStride, 16); break; case 1: QCOMPARE(v.offset, 64); QCOMPARE(v.size, 4); - QCOMPARE(v.name, QLatin1String("opacity")); + QCOMPARE(v.name, QByteArrayLiteral("opacity")); QCOMPARE(v.type, QShaderDescription::Float); break; default: |