diff options
author | Kartik K. Agaram <vc@akkartik.com> | 2016-11-08 10:31:48 -0800 |
---|---|---|
committer | Kartik K. Agaram <vc@akkartik.com> | 2016-11-08 10:31:48 -0800 |
commit | 4ecab1821ed7609d35f2c9509cb045dc79606e8c (patch) | |
tree | c5f7f3641fd24fac4c36139a81fcddd4dab188e5 /036refcount.cc | |
parent | f4647409b5fa658e463424c0a8249a4d72405d59 (diff) | |
download | mu-4ecab1821ed7609d35f2c9509cb045dc79606e8c.tar.gz |
3653
Don't crash on bad types. I need to be more careful in distinguishing between the two causes of constraint violations: bad input and internal bugs. Maybe I should create a second assert() to indicate "this shouldn't really be an assert, but I'm too lazy to think about it right now."
Diffstat (limited to '036refcount.cc')
-rw-r--r-- | 036refcount.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/036refcount.cc b/036refcount.cc index 36aea6c0..09bb5d30 100644 --- a/036refcount.cc +++ b/036refcount.cc @@ -311,7 +311,10 @@ void compute_container_address_offsets(reagent& r, const string& location_for_er void compute_container_address_offsets(const type_tree* type, const string& location_for_error_messages) { if (!type) return; if (!type->atom) { - assert(type->left->atom); + if (!type->left->atom) { + raise << "invalid type " << to_string(type) << location_for_error_messages << '\n' << end(); + return; + } if (type->left->name == "address") compute_container_address_offsets(type->right, location_for_error_messages); else if (type->left->name == "array") |