diff options
author | Morgan Deters <mdeters@gmail.com> | 2009-12-16 23:30:21 +0000 |
---|---|---|
committer | Morgan Deters <mdeters@gmail.com> | 2009-12-16 23:30:21 +0000 |
commit | 426b8722e6e32f7fab46769e4d71184bf510fd0e (patch) | |
tree | 8a6fa139a7a599c5a659100df12f0efdc838028d /src/expr/node_manager.cpp | |
parent | b33a35b720059370ffb1507f31603cdb5347503a (diff) |
+ refactoring fixes for expr package based on code review (see bug #4)
+ minor autogen/configure fixes for old versions of autotools
Diffstat (limited to 'src/expr/node_manager.cpp')
-rw-r--r-- | src/expr/node_manager.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/expr/node_manager.cpp b/src/expr/node_manager.cpp index ab52b9f40..4b550ee3d 100644 --- a/src/expr/node_manager.cpp +++ b/src/expr/node_manager.cpp @@ -20,6 +20,8 @@ __thread NodeManager* NodeManager::s_current = 0; Node NodeManager::lookup(uint64_t hash, NodeValue* ev) { Assert(this != NULL, "Whoops, we have a bad expression manager!"); + Assert(ev != NULL, "lookup() expects a non-NULL NodeValue!"); + hash_t::iterator i = d_hash.find(hash); if(i == d_hash.end()) { // insert @@ -63,6 +65,8 @@ Node NodeManager::lookup(uint64_t hash, NodeValue* ev) { NodeValue* NodeManager::lookupNoInsert(uint64_t hash, NodeValue* ev) { Assert(this != NULL, "Whoops, we have a bad expression manager!"); + Assert(ev != NULL, "lookupNoInsert() expects a non-NULL NodeValue!"); + hash_t::iterator i = d_hash.find(hash); if(i == d_hash.end()) { return NULL; |