Fix some copy instead of move warnings

qgis · Dec 15, 2024 · e4bb317 · e4bb317
1 parent c343317
commit e4bb317
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/src/core/pointcloud/qgseptpointcloudindex.cpp b/src/core/pointcloud/qgseptpointcloudindex.cpp
@@ -522,7 +522,7 @@ QgsPointCloudStatistics QgsEptPointCloudIndex::metadataStatistics() const
 
     s.classCount = mAttributeClasses[ name ];
 
-    statsMap[ name ] = s;
+    statsMap[ name ] = std::move( s );
   }
   return QgsPointCloudStatistics( pointCount(), statsMap );
 }

diff --git a/src/core/pointcloud/qgspointcloudstatscalculator.cpp b/src/core/pointcloud/qgspointcloudstatscalculator.cpp
@@ -112,7 +112,7 @@ struct StatsProcessor
         summary.mean = 0;
         summary.stDev = std::numeric_limits<double>::quiet_NaN();
         summary.classCount.clear();
-        statsMap[ attribute.name() ] = summary;
+        statsMap[ attribute.name() ] = std::move( summary );
       }
 
       QVector<int> attributeOffsetVector;

diff --git a/src/core/raster/qgsrasterlayer.cpp b/src/core/raster/qgsrasterlayer.cpp
@@ -1407,8 +1407,7 @@ void QgsRasterLayer::setContrastEnhancement( QgsContrastEnhancement::ContrastEnh
     return;
   }
 
-  const auto constMyBands = myBands;
-  for ( const int myBand : constMyBands )
+  for ( const int myBand : std::as_const( myBands ) )
   {
     if ( myBand != -1 )
     {