Fixed error with Rice 4.11

Author: ankane

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.17.1 (unreleased)
+
+- Fixed error with Rice 4.11
+
 ## 0.17.0 (2026-01-12)
 
 - Updated OR-Tools to 9.15

ext/or-tools/constraint.cpp

@@ -98,8 +98,8 @@ void init_constraint(Rice::Module& m) {
           auto a = Array(literal);
           std::vector<BoolVar> vec;
           vec.reserve(a.size());
-          for (const Object v : a) {
-            if (v.is_a(rb_cSatIntVar)) {
+          for (const auto& v : a) {
+            if (Object(v).is_a(rb_cSatIntVar)) {
               vec.push_back(Rice::detail::From_Ruby<IntVar>().convert(v.value()).ToBoolVar());
             } else {
               vec.push_back(Rice::detail::From_Ruby<BoolVar>().convert(v.value()));

ext/or-tools/linear.cpp

@@ -1,4 +1,5 @@
 #include <memory>
+#include <optional>
 #include <string>
 
 #include <ortools/linear_solver/linear_solver.h>
@@ -114,11 +115,11 @@ void init_linear(Rice::Module& m) {
       [](MPSolverParameters& self) {
         int presolve = self.GetIntegerParam(MPSolverParameters::IntegerParam::PRESOLVE);
         if (presolve == MPSolverParameters::PresolveValues::PRESOLVE_ON) {
-          return Rice::True;
+          return std::optional<bool>{true};
         } else if (presolve == MPSolverParameters::PresolveValues::PRESOLVE_OFF) {
-          return Rice::False;
+          return std::optional<bool>{false};
         } else {
-          return Rice::Nil;
+          return std::optional<bool>{};
         }
       })
     .define_method(
@@ -137,11 +138,11 @@ void init_linear(Rice::Module& m) {
       [](MPSolverParameters& self) {
         int incrementality = self.GetIntegerParam(MPSolverParameters::IntegerParam::INCREMENTALITY);
         if (incrementality == MPSolverParameters::IncrementalityValues::INCREMENTALITY_ON) {
-          return Rice::True;
+          return std::optional<bool>{true};
         } else if (incrementality == MPSolverParameters::IncrementalityValues::INCREMENTALITY_OFF) {
-          return Rice::False;
+          return std::optional<bool>{false};
         } else {
-          return Rice::Nil;
+          return std::optional<bool>{};
         }
       })
     .define_method(
@@ -160,11 +161,11 @@ void init_linear(Rice::Module& m) {
       [](MPSolverParameters& self) {
         int scaling = self.GetIntegerParam(MPSolverParameters::IntegerParam::SCALING);
         if (scaling == MPSolverParameters::ScalingValues::SCALING_ON) {
-          return Rice::True;
+          return std::optional<bool>{true};
         } else if (scaling == MPSolverParameters::ScalingValues::SCALING_OFF) {
-          return Rice::False;
+          return std::optional<bool>{false};
         } else {
-          return Rice::Nil;
+          return std::optional<bool>{};
         }
       });
 

ext/or-tools/routing.cpp

@@ -295,7 +295,7 @@ void init_routing(Rice::Module& m) {
       "cumulative",
       [](operations_research::Solver& self, Array rb_intervals, std::vector<int64_t> demands, int64_t capacity, const std::string& name) {
         std::vector<operations_research::IntervalVar*> intervals;
-        for (const Object v : rb_intervals) {
+        for (const auto& v : rb_intervals) {
           intervals.push_back(Rice::detail::From_Ruby<operations_research::IntervalVar*>().convert(v.value()));
         }
         return self.MakeCumulative(intervals, demands, capacity, name);