Rather than just aborting, report an intelligent error if the comma operator

is missing one of its operands in a value expression.  This kind of reporting
still needs to be done for all the other operators as well.

1 files changed, 10 insertions, 4 deletions

diff --git a/valexpr.cc b/valexpr.cc
index 6afd0805..5b5d29ba 100644
--- a/valexpr.cc
+++ b/valexpr.cc
@@ -578,8 +578,12 @@ void value_expr_t::compute(value_t& result, const details_t& details,
   }
 
   case O_COM:
-    assert(left);
-    assert(right);
+    if (! left)
+      throw new compute_error("Comma operator missing left operand",
+			      new valexpr_context(this));
+    if (! right)
+      throw new compute_error("Comma operator missing right operand",
+			      new valexpr_context(this));
     left->compute(result, details, context);
     right->compute(result, details, context);
     break;
@@ -1731,10 +1735,12 @@ bool write_value_expr(std::ostream&	   out,
     break;
 
   case value_expr_t::O_COM:
-    if (write_value_expr(out, node->left, relaxed, node_to_find, start_pos, end_pos))
+    if (node->left &&
+	write_value_expr(out, node->left, relaxed, node_to_find, start_pos, end_pos))
       found = true;
     out << ", ";
-    if (write_value_expr(out, node->right, relaxed, node_to_find, start_pos, end_pos))
+    if (node->right &&
+	write_value_expr(out, node->right, relaxed, node_to_find, start_pos, end_pos))
       found = true;
     break;
   case value_expr_t::O_QUES: