A mess of changes in the expression manager, simple example still failing due to...
authorDejan Jovanović <dejan.jovanovic@gmail.com>
Wed, 9 Dec 2009 03:36:52 +0000 (03:36 +0000)
committerDejan Jovanović <dejan.jovanovic@gmail.com>
Wed, 9 Dec 2009 03:36:52 +0000 (03:36 +0000)
commitd697d1e91be226339a28bd7e8dce3862901cba8a
tree9cb747bfd29ffe4e5b470bda05653ee0a27ad166
parent0fcbc89e92a1137f6829a4a59138fc20c43d194d
A mess of changes in the expression manager, simple example still failing due to some problems in the prop_engine
* default null expr and expr value and reorganisation/rewrite of some methods
* fixed some bugs
* expressions should always be passed as const Expr& to methods, otherwise copy constructors are called

Problematic code:
* Expr class has a bunch of methods that return Exprs, such as a.andExpr(b). None of these know what is their expression manager. We should
 (a) Not allow this, all expression construction should go through the ExprBuilder or directly thorugh the expression manager
 (b) Allow this, as it is now, but the have the default expression manager be setup in every entry into the smt solver + these construction methods shouldn't be available to the user (oterwise a mess ensues)
* We have to decide how to construct ExprBuilders:
 (a) constructing ExprBuilders with em = ExprBuilder(e).andExpr(b) is problematic as at construction we can't know the expression manager, and the kind of em will be UNDEFINED, so when adding b we can't assume its not UNDEFINED
 (b) constructing it with ExprBuilder(em) << AND << a << b or ExprBuilder(em, AND) << a << b seems like a nicer approach

 I am still confused about these expression builders so we should talk about this.
src/expr/expr.cpp
src/expr/expr.h
src/expr/expr_builder.cpp
src/expr/expr_builder.h
src/expr/expr_manager.cpp
src/expr/expr_manager.h
src/expr/expr_value.cpp
src/expr/expr_value.h
src/smt/smt_engine.cpp
src/util/command.cpp
src/util/command.h