From a31749966aa4c19a1c94dd8684fffdeea9b5baa8 Mon Sep 17 00:00:00 2001 From: trevor_hansen Date: Mon, 11 Jan 2010 15:39:52 +0000 Subject: [PATCH] Bugfix. Simplifying minisat not setting a return code as expected. Copied from my branch r517. git-svn-id: https://stp-fast-prover.svn.sourceforge.net/svnroot/stp-fast-prover/trunk/stp@552 e59a4935-1847-0410-ae03-e826735625c1 --- src/sat/simp/SimpSolver.C | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/sat/simp/SimpSolver.C b/src/sat/simp/SimpSolver.C index 03b1990..4a3cf10 100644 --- a/src/sat/simp/SimpSolver.C +++ b/src/sat/simp/SimpSolver.C @@ -110,6 +110,13 @@ bool SimpSolver::solve(const vec& assumps, bool do_simp, bool turn_off_simp for (int i = 0; i < extra_frozen.size(); i++) setFrozen(extra_frozen[i], false); + // STP MODIFICATION.. + // If eliminate(bool) finds that the formula is unsatisfiable. ok may not be set. + // We use okay() to check if the function was satisfiable. + if (!result) + ok=false; + // STP MODIFICATION.. + return result; } -- 2.47.3