🚨 fix a couple of gcc compiler warnings

Signed-off-by: burgholzer <[email protected]>

Author: burgholzer

src/logicblocks/LogicTerm.cpp

@@ -334,7 +334,7 @@ int LogicTerm::getIntValue() const {
   case CType::BITVECTOR:
     return static_cast<int>(bvValue);
   default:
-    return std::numeric_limits<int>::infinity();
+    return std::numeric_limits<int>::max();
   }
 }
 
@@ -349,7 +349,7 @@ double LogicTerm::getFloatValue() const {
   case CType::BITVECTOR:
     return static_cast<double>(bvValue);
   default:
-    return std::numeric_limits<double>::infinity();
+    return std::numeric_limits<double>::max();
   }
 }
 
@@ -364,7 +364,7 @@ uint64_t LogicTerm::getBitVectorValue() const {
   case CType::BITVECTOR:
     return bvValue & (static_cast<uint64_t>(std::pow(2, bvSize)) - 1U);
   default:
-    return std::numeric_limits<uint64_t>::infinity();
+    return std::numeric_limits<uint64_t>::max();
   }
 }
 
@@ -379,7 +379,7 @@ uint16_t LogicTerm::getBitVectorSize() const {
   case CType::BITVECTOR:
     return bvSize;
   default:
-    return std::numeric_limits<uint16_t>::infinity();
+    return std::numeric_limits<uint16_t>::max();
   }
 }
 

test/sc/heuristic/test_heuristic.cpp

@@ -1299,33 +1299,21 @@ TEST(Functionality, DataLogger) {
   const auto& singleQubitFidelityCosts =
       architecture.getSingleQubitFidelityCosts();
   for (std::size_t i = 0; i < singleQubitFidelityCosts.size(); ++i) {
-    if (std::isinf(singleQubitFidelityCosts[i])) {
-      EXPECT_TRUE(fidelityJson["single_qubit_fidelity_costs"][i].is_null());
-    } else {
-      EXPECT_EQ(fidelityJson["single_qubit_fidelity_costs"][i],
-                singleQubitFidelityCosts[i]);
-    }
+    EXPECT_EQ(fidelityJson["single_qubit_fidelity_costs"][i],
+              singleQubitFidelityCosts[i]);
   }
   const auto& twoQubitFidelityCosts = architecture.getTwoQubitFidelityCosts();
   for (std::size_t i = 0; i < twoQubitFidelityCosts.size(); ++i) {
     for (std::size_t j = 0; j < twoQubitFidelityCosts[i].size(); ++j) {
-      if (std::isinf(twoQubitFidelityCosts[i][j])) {
-        EXPECT_TRUE(fidelityJson["two_qubit_fidelity_costs"][i][j].is_null());
-      } else {
-        EXPECT_EQ(fidelityJson["two_qubit_fidelity_costs"][i][j],
-                  twoQubitFidelityCosts[i][j]);
-      }
+      EXPECT_EQ(fidelityJson["two_qubit_fidelity_costs"][i][j],
+                twoQubitFidelityCosts[i][j]);
     }
   }
   const auto& swapFidelityCosts = architecture.getSwapFidelityCosts();
   for (std::size_t i = 0; i < swapFidelityCosts.size(); ++i) {
     for (std::size_t j = 0; j < swapFidelityCosts[i].size(); ++j) {
-      if (std::isinf(swapFidelityCosts[i][j])) {
-        EXPECT_TRUE(fidelityJson["swap_fidelity_costs"][i][j].is_null());
-      } else {
-        EXPECT_EQ(fidelityJson["swap_fidelity_costs"][i][j],
-                  swapFidelityCosts[i][j]);
-      }
+      EXPECT_EQ(fidelityJson["swap_fidelity_costs"][i][j],
+                swapFidelityCosts[i][j]);
     }
   }