Insert closure calls in conditional expression of assert (issue 1584).

runtime/vm/ast.h

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef VM_AST_H_
 #define VM_AST_H_
 
 #include "platform/assert.h"
 #include "vm/allocation.h"
 #include "vm/growable_array.h"
@@ skipping 640 matching lines @@
         false_expr_(false_expr) {
       ASSERT(condition_ != NULL);
       ASSERT(true_expr_ != NULL);
       ASSERT(false_expr_ != NULL);
   }
 
   AstNode* condition() const { return condition_; }
   AstNode* true_expr() const { return true_expr_; }
   AstNode* false_expr() const { return false_expr_; }
 
+  void set_true_expr(AstNode* true_expr) {
+    ASSERT(true_expr_ != NULL);

[srdjan, 2012/05/31 15:39:11]

Why are these asserts necessary? Dop you want to check that true_expr != NULL
instead (also below)

[regis, 2012/05/31 15:56:24]

Yes, of course. Copy and paste error. Thanks!

+    true_expr_ = true_expr;
+  }
+  void set_false_expr(AstNode* false_expr) {
+    ASSERT(false_expr_ != NULL);
+    false_expr_ = false_expr;
+  }
+
   virtual void VisitChildren(AstNodeVisitor* visitor) const {
     condition()->Visit(visitor);
     true_expr()->Visit(visitor);
     false_expr()->Visit(visitor);
   }
 
   DECLARE_COMMON_NODE_FUNCTIONS(ConditionalExprNode);
 
  private:
   AstNode* condition_;
@@ skipping 990 matching lines @@
   const LocalVariable& context_var_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(InlinedFinallyNode);
 };
 
 }  // namespace dart
 
 #undef DECLARE_COMMON_NODE_FUNCTIONS
 
 #endif  // VM_AST_H_