Fixed formatting issues and unit test (-1 is not a valid enum value).

Author: mathgeekcoder

check/TestCAPI.c

@@ -126,7 +126,7 @@ static void userCallback(const int callback_type, const char* message,
       printf("userCallback(%11.4g): improving solution with objective = %g\n",
              local_callback_data, objective_function_value);
     // Now test all more simply
-    checkGetCallbackDataOutInt(data_out, kHighsCallbackDataOutLogTypeName, -1);
+    checkGetCallbackDataOutInt(data_out, kHighsCallbackDataOutLogTypeName, 0);
     checkGetCallbackDataOutDouble(
         data_out, kHighsCallbackDataOutRunningTimeName, data_out->running_time);
     checkGetCallbackDataOutHighsInt(

src/lp_data/Highs.cpp

@@ -2226,19 +2226,19 @@ HighsStatus Highs::setCallback(HighsCallbackFunctionType user_callback,
 HighsStatus Highs::setCallback(HighsCCallbackType c_callback,
                                void* user_callback_data) {
   this->callback_.clear();
-  this->callback_.user_callback =
-      [c_callback](int a, const std::string& b, const HighsCallbackDataOut* cb_out,
-                   HighsCallbackDataIn* cb_in,
-                   void* e) { 
-        HighsCCallbackDataOut cc_out = static_cast<HighsCCallbackDataOut>(*cb_out);
-        HighsCCallbackDataIn cc_in;
-        cc_in.user_interrupt = 0;
-        cc_in.user_solution_size = 0;
-        cc_in.user_solution = nullptr;
-
-        c_callback(a, b.c_str(), &cc_out, &cc_in, e); 
-        *cb_in = cc_in;  // copy the data in
-      };
+  this->callback_.user_callback = [c_callback](
+                                      int a, const std::string& b,
+                                      const HighsCallbackDataOut* cb_out,
+                                      HighsCallbackDataIn* cb_in, void* e) {
+    HighsCCallbackDataOut cc_out = static_cast<HighsCCallbackDataOut>(*cb_out);
+    HighsCCallbackDataIn cc_in;
+    cc_in.user_interrupt = 0;
+    cc_in.user_solution_size = 0;
+    cc_in.user_solution = nullptr;
+
+    c_callback(a, b.c_str(), &cc_out, &cc_in, e);
+    *cb_in = cc_in;  // copy the data in
+  };
   this->callback_.user_callback_data = user_callback_data;
 
   options_.log_options.user_callback = this->callback_.user_callback;

src/mip/HighsMipSolverData.cpp

@@ -2504,7 +2504,7 @@ void HighsMipSolverData::callbackUserSolution(
   assert(!interrupt);
   if (mipsolver.callback_->data_in.user_solution.size() ==
       mipsolver.orig_model_->num_col_) {
-    const std::vector<double>& user_solution = mipsolver.callback_->data_in.user_solution;
+    const auto& user_solution = mipsolver.callback_->data_in.user_solution;
     double bound_violation_ = 0;
     double row_violation_ = 0;
     double integrality_violation_ = 0;