try and be a bit safer at end of dual (partly for TwoMir)

src/ClpSimplex.cpp

@@ -5588,7 +5588,6 @@ int ClpSimplex::dualDebug(int ifValuesPass, int startFinishOptions)
 
          This is to stop classes becoming too unwieldy and so I (JJF) can use e.g. "perturb"
          in primal and dual.
-
          As far as I can see this is perfectly safe.
      */
 #ifdef COIN_DEVELOP
@@ -5683,7 +5682,8 @@ int ClpSimplex::dualDebug(int ifValuesPass, int startFinishOptions)
 #endif
   //int lastAlgorithm = -1;
   if ((specialOptions_ & 2048) != 0 && problemStatus_ == 10 && !numberPrimalInfeasibilities_
-    && sumDualInfeasibilities_ < 50.0 * dualTolerance_ && perturbation_ >= 100)
+      && sumDualInfeasibilities_ < 50.0 * dualTolerance_
+      && perturbation_ >= 100 && objectiveValue_ >= bestObjectiveValue_)
     problemStatus_ = 0; // ignore
   if (problemStatus_ == 1 && ((specialOptions_ & (1024 | 4096)) == 0 || (specialOptions_ & 32) != 0)
     && (static_cast< ClpSimplexDual * >(this))->checkFakeBounds()) {

src/ClpSimplexDual.cpp

@@ -327,6 +327,8 @@ int ClpSimplexDual::startupSolve(int ifValuesPass, double *saveDuals, int startF
         usePrimal = perturb();
       // Can't get here if values pass
       gutsOfSolution(NULL, NULL);
+      // save for later check
+      bestObjectiveValue_=objectiveValue_;
 #ifdef CLP_INVESTIGATE
       if (numberDualInfeasibilities_)
         printf("ZZZ %d primal %d dual - sumdinf %g\n",