diff options
author | Sebastian Parborg <darkdefende@gmail.com> | 2020-09-04 21:59:13 +0300 |
---|---|---|
committer | Sebastian Parborg <darkdefende@gmail.com> | 2020-09-04 22:04:16 +0300 |
commit | 2115232a16d81d28dbdb8042ed8e9316858514c6 (patch) | |
tree | 1aeb7354a85b21b43a3ede7bf2980c172d4eec82 /source/blender/blenlib/tests | |
parent | e43d482cc93c64d55b1f58178db68551077e6560 (diff) |
Cleanup: Clang-Tidy readability-inconsistent-declaration-parameter-name fix
No functional changes
Diffstat (limited to 'source/blender/blenlib/tests')
-rw-r--r-- | source/blender/blenlib/tests/BLI_edgehash_test.cc | 6 | ||||
-rw-r--r-- | source/blender/blenlib/tests/BLI_string_test.cc | 8 |
2 files changed, 7 insertions, 7 deletions
diff --git a/source/blender/blenlib/tests/BLI_edgehash_test.cc b/source/blender/blenlib/tests/BLI_edgehash_test.cc index 7106033df36..f9f0f99daa8 100644 --- a/source/blender/blenlib/tests/BLI_edgehash_test.cc +++ b/source/blender/blenlib/tests/BLI_edgehash_test.cc @@ -242,11 +242,11 @@ TEST(edgehash, IteratorFindsAllValues) BLI_edgehash_insert(eh, 1, 4, VALUE_3); EdgeHashIterator *ehi = BLI_edgehashIterator_new(eh); - auto a = BLI_edgehashIterator_getValue(ehi); + auto *a = BLI_edgehashIterator_getValue(ehi); BLI_edgehashIterator_step(ehi); - auto b = BLI_edgehashIterator_getValue(ehi); + auto *b = BLI_edgehashIterator_getValue(ehi); BLI_edgehashIterator_step(ehi); - auto c = BLI_edgehashIterator_getValue(ehi); + auto *c = BLI_edgehashIterator_getValue(ehi); BLI_edgehashIterator_step(ehi); ASSERT_NE(a, b); diff --git a/source/blender/blenlib/tests/BLI_string_test.cc b/source/blender/blenlib/tests/BLI_string_test.cc index a5fd3e31c31..12f0fd5ac62 100644 --- a/source/blender/blenlib/tests/BLI_string_test.cc +++ b/source/blender/blenlib/tests/BLI_string_test.cc @@ -608,21 +608,21 @@ class StringCasecmpNatural : public testing::Test { void testReturnsZeroForAll(const CompareWordsArray &items) { - for (auto &item : items) { + for (const auto &item : items) { int res = BLI_strcasecmp_natural(item[0], item[1]); EXPECT_EQ(res, 0); } } void testReturnsLessThanZeroForAll(const CompareWordsArray &items) { - for (auto &item : items) { + for (const auto &item : items) { int res = BLI_strcasecmp_natural(item[0], item[1]); EXPECT_LT(res, 0); } } void testReturnsMoreThanZeroForAll(const CompareWordsArray &items) { - for (auto &item : items) { + for (const auto &item : items) { int res = BLI_strcasecmp_natural(item[0], item[1]); EXPECT_GT(res, 0); } @@ -635,7 +635,7 @@ class StringCasecmpNatural : public testing::Test { /* E.g. {{"a", "b"}, {"ab", "cd"}} becomes {{"b", "a"}, {"cd", "ab"}} */ ret_array.reserve(items.size()); - for (auto &item : items) { + for (const auto &item : items) { ret_array.push_back({item[1], item[0]}); } |