Fix EqProof to ProofNode conversion (#4760)
authorHaniel Barbosa <hanielbbarbosa@gmail.com>
Fri, 17 Jul 2020 02:23:05 +0000 (23:23 -0300)
committerGitHub <noreply@github.com>
Fri, 17 Jul 2020 02:23:05 +0000 (21:23 -0500)
A wrong change slipped away during the cleaning of the module. This commit fixes the conversion.

src/theory/uf/eq_proof.cpp

index 70edbd0dd52c76265a4b4410e775e01857a1e64e..c7a834b1903a1495e2c741d5e090df15209fabad 100644 (file)
@@ -997,7 +997,6 @@ Node EqProof::addToProof(
       if (childProof->d_id == MERGED_THROUGH_CONGRUENCE
           && childProof->d_node.isNull())
       {
-        CVC4_UNUSED bool addedChild = false;
         Trace("eqproof-conv") << "EqProof::addToProof: child proof " << i
                               << " is fake cong step. Fold it.\n";
         Assert(childProof->d_children.size() == 2);
@@ -1008,12 +1007,11 @@ Node EqProof::addToProof(
           Trace("eqproof-conv")
               << "EqProof::addToProof: recurse on child " << j << "\n"
               << push;
-          Node child =
-              childProof->d_children[j]->addToProof(p, visited, assumptions);
+          children.push_back(
+              childProof->d_children[j]->addToProof(p, visited, assumptions));
           Trace("eqproof-conv") << pop;
         }
         Trace("eqproof-conv") << pop;
-        Assert(addedChild);
         continue;
       }
       Trace("eqproof-conv")