qDebug & QString cleanup

NEDJIMAbelgacem · NEDJIMAbelgacem · commit 752a27167416 · 2020-07-22T17:46:57.000+01:00
diff --git a/src/3d/qgs3dexportobject.cpp b/src/3d/qgs3dexportobject.cpp
@@ -88,10 +88,10 @@ void Qgs3DExportObject::setupPhongMaterial( const QgsPhongMaterialSettings &mate
   QColor specular = material.specular();
   QColor ambient = material.ambient();
   float shininess = material.shininess();
-  setMaterialParameter( QString( "Kd" ), QString( "%1 %2 %3" ).arg( diffuse.redF() ).arg( diffuse.greenF() ).arg( diffuse.blueF() ) );
-  setMaterialParameter( QString( "Ka" ), QString( "%1 %2 %3" ).arg( ambient.redF() ).arg( ambient.greenF() ).arg( ambient.blueF() ) );
-  setMaterialParameter( QString( "Ks" ), QString( "%1 %2 %3" ).arg( specular.redF() ).arg( specular.greenF() ).arg( specular.blueF() ) );
-  setMaterialParameter( QString( "Ns" ), QString( "%1" ).arg( shininess ) );
+  setMaterialParameter( QStringLiteral( "Kd" ), QStringLiteral( "%1 %2 %3" ).arg( diffuse.redF() ).arg( diffuse.greenF() ).arg( diffuse.blueF() ) );
+  setMaterialParameter( QStringLiteral( "Ka" ), QStringLiteral( "%1 %2 %3" ).arg( ambient.redF() ).arg( ambient.greenF() ).arg( ambient.blueF() ) );
+  setMaterialParameter( QStringLiteral( "Ks" ), QStringLiteral( "%1 %2 %3" ).arg( specular.redF() ).arg( specular.greenF() ).arg( specular.blueF() ) );
+  setMaterialParameter( QStringLiteral( "Ns" ), QStringLiteral( "%1" ).arg( shininess ) );
 }
 
 void Qgs3DExportObject::objectBounds( float &minX, float &minY, float &minZ, float &maxX, float &maxY, float &maxZ )
@@ -154,12 +154,12 @@ void Qgs3DExportObject::saveTo( QTextStream &out, float scale, const QVector3D &
   {
     int negativeIndex = -1 - ( verticesCount - i );
     if ( hasNormals && !hasTextures )
-      return QString( "%1//%2" ).arg( negativeIndex ).arg( negativeIndex );
+      return QStringLiteral( "%1//%2" ).arg( negativeIndex ).arg( negativeIndex );
     if ( !hasNormals && hasTextures )
-      return QString( "%1/%2" ).arg( negativeIndex ).arg( negativeIndex );
+      return QStringLiteral( "%1/%2" ).arg( negativeIndex ).arg( negativeIndex );
     if ( hasNormals && hasTextures )
-      return QString( "%1/%2/%3" ).arg( negativeIndex ).arg( negativeIndex ).arg( negativeIndex );
-    return QString( "%1" ).arg( negativeIndex );
+      return QStringLiteral( "%1/%2/%3" ).arg( negativeIndex ).arg( negativeIndex ).arg( negativeIndex );
+    return QStringLiteral( "%1" ).arg( negativeIndex );
   };
 
   if ( mType == TriangularFaces )
diff --git a/src/3d/qgs3dsceneexporter.cpp b/src/3d/qgs3dsceneexporter.cpp
@@ -336,7 +336,6 @@ void Qgs3DSceneExporter::processEntityMaterial( Qt3DCore::QEntity *entity, Qgs3D
     }
     if ( imageTexture != nullptr )
     {
-      qDebug() << "Exporting image texture";
       QImage image = imageTexture->getImage();
       object->setTextureImage( image );
     }
@@ -652,12 +651,6 @@ Qgs3DExportObject *Qgs3DSceneExporter::processGeometryRenderer( Qt3DRender::QGeo
     return nullptr;
   }
 
-  if ( positionData.size() == 0 )
-  {
-    qDebug() << "positionData.size()==" << positionData.size();
-    qDebug() << "indexData.size()==" << indexData.size();
-  }
-
   Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( QStringLiteral( "mesh_geometry" ) ), "", this );
   object->setupPositionCoordinates( positionData, scale * sceneScale, translation + sceneTranslation );
   object->setupFaces( indexData );
@@ -792,7 +785,7 @@ QString Qgs3DSceneExporter::getObjectName( const QString &name )
   QString ret = name;
   if ( usedObjectNamesCounter.contains( name ) )
   {
-    ret = QString( "%1%2" ).arg( name ).arg( usedObjectNamesCounter[name] );
+    ret = QStringLiteral( "%1%2" ).arg( name ).arg( usedObjectNamesCounter[name] );
     usedObjectNamesCounter[name]++;
   }
   else

Original file line number	Diff line number	Diff line change
`@@ -336,7 +336,6 @@ void Qgs3DSceneExporter::processEntityMaterial( Qt3DCore::QEntity *entity, Qgs3D`
`336`	`336`	`}`
`337`	`337`	`if ( imageTexture != nullptr )`
`338`	`338`	`{`
`339`		`- qDebug() << "Exporting image texture";`
`340`	`339`	`QImage image = imageTexture->getImage();`
`341`	`340`	`object->setTextureImage( image );`
`342`	`341`	`}`
`@@ -652,12 +651,6 @@ Qgs3DExportObject *Qgs3DSceneExporter::processGeometryRenderer( Qt3DRender::QGeo`
`652`	`651`	`return nullptr;`
`653`	`652`	`}`
`654`	`653`
`655`		`- if ( positionData.size() == 0 )`
`656`		`- {`
`657`		`- qDebug() << "positionData.size()==" << positionData.size();`
`658`		`- qDebug() << "indexData.size()==" << indexData.size();`
`659`		`- }`
`660`		`-`
`661`	`654`	`Qgs3DExportObject *object = new Qgs3DExportObject( getObjectName( QStringLiteral( "mesh_geometry" ) ), "", this );`
`662`	`655`	`object->setupPositionCoordinates( positionData, scale * sceneScale, translation + sceneTranslation );`
`663`	`656`	`object->setupFaces( indexData );`
`@@ -792,7 +785,7 @@ QString Qgs3DSceneExporter::getObjectName( const QString &name )`
`792`	`785`	`QString ret = name;`
`793`	`786`	`if ( usedObjectNamesCounter.contains( name ) )`
`794`	`787`	`{`
`795`		`- ret = QString( "%1%2" ).arg( name ).arg( usedObjectNamesCounter[name] );`
	`788`	`+ ret = QStringLiteral( "%1%2" ).arg( name ).arg( usedObjectNamesCounter[name] );`
`796`	`789`	`usedObjectNamesCounter[name]++;`
`797`	`790`	`}`
`798`	`791`	`else`