Merge pull request #2495 from ERGO-Code/warnings-gcc

Warnings g++ linux

Author: jajhall

highs/interfaces/highs_c_api.cpp

@@ -1320,32 +1320,34 @@ HighsInt Highs_getIis(void* highs, HighsInt* iis_num_col, HighsInt* iis_num_row,
   *iis_num_col = iis.col_index_.size();
   *iis_num_row = iis.row_index_.size();
   if (col_index != nullptr) {
-    for (size_t i = 0; i < *iis_num_col; i++) {
+    for (size_t i = 0; i < static_cast<size_t>(*iis_num_col); i++) {
       col_index[i] = iis.col_index_[i];
     }
   }
   if (row_index != nullptr) {
-    for (size_t i = 0; i < *iis_num_row; i++) {
+    for (size_t i = 0; i < static_cast<size_t>(*iis_num_row); i++) {
       row_index[i] = iis.row_index_[i];
     }
   }
   if (col_bound != nullptr) {
-    for (size_t i = 0; i < *iis_num_col; i++) {
+    for (size_t i = 0; i < static_cast<size_t>(*iis_num_col); i++) {
       col_bound[i] = iis.col_bound_[i];
     }
   }
   if (row_bound != nullptr) {
-    for (size_t i = 0; i < *iis_num_row; i++) {
+    for (size_t i = 0; i < static_cast<size_t>(*iis_num_row); i++) {
       row_bound[i] = iis.row_bound_[i];
     }
   }
   if (col_status != nullptr) {
-    for (size_t i = 0; i < ((Highs*)highs)->getLp().num_col_; i++) {
+    for (size_t i = 0;
+         i < static_cast<size_t>(((Highs*)highs)->getLp().num_col_); i++) {
       col_status[i] = iis.col_status_[i];
     }
   }
   if (row_status != nullptr) {
-    for (size_t i = 0; i < ((Highs*)highs)->getLp().num_row_; i++) {
+    for (size_t i = 0;
+         i < static_cast<size_t>(((Highs*)highs)->getLp().num_row_); i++) {
       row_status[i] = iis.row_status_[i];
     }
   }

highs/lp_data/HighsLp.cpp

@@ -18,7 +18,7 @@
 bool HighsLp::isMip() const {
   size_t integrality_size = this->integrality_.size();
   if (integrality_size) {
-    assert(integrality_size == this->num_col_);
+    assert(static_cast<HighsInt>(integrality_size) == this->num_col_);
     for (HighsInt iCol = 0; iCol < this->num_col_; iCol++)
       if (this->integrality_[iCol] != HighsVarType::kContinuous) return true;
   }
@@ -36,7 +36,7 @@ bool HighsLp::hasInfiniteCost(const double infinite_cost) const {
 bool HighsLp::hasSemiVariables() const {
   size_t integrality_size = this->integrality_.size();
   if (integrality_size) {
-    assert(integrality_size == this->num_col_);
+    assert(static_cast<HighsInt>(integrality_size) == this->num_col_);
     for (HighsInt iCol = 0; iCol < this->num_col_; iCol++)
       if (this->integrality_[iCol] == HighsVarType::kSemiContinuous ||
           this->integrality_[iCol] == HighsVarType::kSemiInteger)