Public interface review items:

* Internal uses of CheckArgument changed to AssertArgument/AlwaysAssertArgument()
* Make util/Assert.h cvc4_private instead of public, so AssertionException and friends are now internal-only
* CheckArgument() throws non-AssertionException
* things outside the core library (parsers, driver) use regular C-style assert,
  or a public exception type.
* auto-generated documentation for Smt options and internal options

Also, a small fix to SMT-LIBv1 QF_ABV and QF_AUFBV definitions, which were nonstandard.

1 files changed, 6 insertions, 6 deletions

diff --git a/src/prop/cnf_stream.cpp b/src/prop/cnf_stream.cpp
index c5345c5a7..9fb825e47 100644
--- a/src/prop/cnf_stream.cpp
+++ b/src/prop/cnf_stream.cpp
@@ -134,12 +134,12 @@ void TseitinCnfStream::ensureLiteral(TNode n) {
     return;
   }
 
-  CheckArgument(n.getType().isBoolean(), n,
-                "CnfStream::ensureLiteral() requires a node of Boolean type.\n"
-                "got node: %s\n"
-                "its type: %s\n",
-                n.toString().c_str(),
-                n.getType().toString().c_str());
+  AlwaysAssertArgument(n.getType().isBoolean(), n,
+                       "CnfStream::ensureLiteral() requires a node of Boolean type.\n"
+                       "got node: %s\n"
+                       "its type: %s\n",
+                       n.toString().c_str(),
+                       n.getType().toString().c_str());
 
   bool negated CVC4_UNUSED = false;
   SatLiteral lit;