[Bf-blender-cvs] [d0b68f9bb22] temp-asset-browser-catalogs: Avoid unnecessary allocations in unit test

Julian Eisel noreply at git.blender.org
Mon Sep 6 15:45:51 CEST 2021


Commit: d0b68f9bb229f9ba4a260fae8331f0cf4608fa0a
Author: Julian Eisel
Date:   Mon Sep 6 15:44:06 2021 +0200
Branches: temp-asset-browser-catalogs
https://developer.blender.org/rBd0b68f9bb229f9ba4a260fae8331f0cf4608fa0a

Avoid unnecessary allocations in unit test

===================================================================

M	source/blender/blenkernel/intern/asset_catalog_test.cc

===================================================================

diff --git a/source/blender/blenkernel/intern/asset_catalog_test.cc b/source/blender/blenkernel/intern/asset_catalog_test.cc
index e547da86ccd..7ec346e439a 100644
--- a/source/blender/blenkernel/intern/asset_catalog_test.cc
+++ b/source/blender/blenkernel/intern/asset_catalog_test.cc
@@ -96,8 +96,9 @@ TEST_F(AssetCatalogTest, load_single_file)
 static int count_path_parents(const fs::path &path)
 {
   int counter = 0;
-  for (fs::path segment : path.parent_path()) {
+  for (const fs::path &segment : path.parent_path()) {
     counter++;
+    UNUSED_VARS(segment);
   }
   return counter;
 }



More information about the Bf-blender-cvs mailing list