summaryrefslogtreecommitdiffstats
path: root/tests/auto/gui/rhi/qshader/tst_qshader.cpp
diff options
context:
space:
mode:
authorJonas Karlsson <jonas.karlsson@qt.io>2020-05-27 12:34:29 +0200
committerJonas Karlsson <jonas.karlsson@qt.io>2020-06-03 18:56:21 +0200
commit1b33ee95e5c6e5e27f732fd273920861fdae486a (patch)
treef8e77cab65d4d32df4f65ee155e01b75e5993598 /tests/auto/gui/rhi/qshader/tst_qshader.cpp
parent1f88ccc8ad64096209cdf7874af129f4220b00ba (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.cpp38
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: