Fix solver_black unit test (#7233)
authorOuyancheng <1024842937@qq.com>
Wed, 22 Sep 2021 19:52:44 +0000 (12:52 -0700)
committerGitHub <noreply@github.com>
Wed, 22 Sep 2021 19:52:44 +0000 (19:52 +0000)
In file `test/unit/api/solver_black.cpp` line 1499,
`for (const std::pair<Term, Term>& t : dmap)` is not the correct way of iterating through the element pairs,
it should be `for (const std::pair<const Term, Term>& t : dmap)` as the keys are immutable.
This triggers a warning on LLVM clang 12.0.1 (not AppleClang) on macOS.

test/unit/api/solver_black.cpp

index fe8cf87baffe63fb12f577a8346d56f6d8299569..63556fa8a814184cb87ed9f05cd9d2bb8df5fbea 100644 (file)
@@ -1496,7 +1496,7 @@ TEST_F(TestApiBlackSolver, getDifficulty3)
   std::map<Term, Term> dmap;
   ASSERT_NO_THROW(dmap = d_solver.getDifficulty());
   // difficulty should map assertions to integer values
-  for (const std::pair<Term, Term>& t : dmap)
+  for (const std::pair<const Term, Term>& t : dmap)
   {
     ASSERT_TRUE(t.first == f0 || t.first == f1);
     ASSERT_TRUE(t.second.getKind() == CONST_RATIONAL);