Moved PDHG_Destroy(&w); to eliminate unassigned read in CupslpWrapper.cpp

Author: jajhall

highs/pdlp/CupdlpWrapper.cpp

@@ -210,6 +210,11 @@ HighsStatus solveLpCupdlp(const HighsOptions& options, HighsTimer& timer,
                             highs_solution.col_value.data(), lp.num_col_,
                             highs_solution.row_dual.data(), lp.num_row_);
   if (w->debug_pdlp_log_file_) fclose(w->debug_pdlp_log_file_);
+  // Moved this from LP_SolvePDHG so w->debug_pdlp_log_file_ can
+  // still be used
+  //
+  PDHG_Destroy(&w);
+
   model_status = HighsModelStatus::kUnknown;
   highs_solution.value_valid = value_valid;
   highs_solution.dual_valid = dual_valid;

highs/pdlp/cupdlp/cupdlp_solver.c

@@ -1476,6 +1476,9 @@ cupdlp_retcode LP_SolvePDHG(
   }
 
 exit_cleanup:
-  PDHG_Destroy(&pdhg);
+  // Move this to CupdlpWrapper so pdhg->debug_pdlp_log_file_ can
+  // still be used
+  //
+  //  PDHG_Destroy(&pdhg);
   return retcode;
 }