From 656004c54655ab15289d9e7666bda2e1c7bada1c Mon Sep 17 00:00:00 2001 From: Andrew Reynolds Date: Tue, 20 Oct 2020 15:20:35 -0500 Subject: (proof-new) Update add lazy step interface in LazyCDProof (#5299) Ensuring closed proofs should not be enabled by default, it is actually not used very often as a whole. Moreover, the "trust id" argument is the most useful argument and hence should come as the 3rd argument. This updates all uses of addLazyStep for the change in interface, also changes term conversion generator which had a similar issue with default arguments. Notice that some calls to addLazyStep were checking closed but without providing a debug string, these I've left alone (they no longer check closed). --- src/smt/term_formula_removal.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'src/smt/term_formula_removal.cpp') diff --git a/src/smt/term_formula_removal.cpp b/src/smt/term_formula_removal.cpp index cf7c00e2b..46135f12a 100644 --- a/src/smt/term_formula_removal.cpp +++ b/src/smt/term_formula_removal.cpp @@ -334,10 +334,9 @@ Node RemoveTermFormulas::runCurrent(std::pair& curr, ProofGenerator* expg = sm->getProofGenerator(existsAssertion); d_lp->addLazyStep(existsAssertion, expg, + PfRule::WITNESS_AXIOM, true, - "RemoveTermFormulas::run:skolem_pf", - false, - PfRule::WITNESS_AXIOM); + "RemoveTermFormulas::run:skolem_pf"); d_lp->addStep(newAssertion, PfRule::SKOLEMIZE, {existsAssertion}, {}); newAssertionPg = d_lp.get(); } -- cgit v1.2.3