- Get rid of RawClosure class and use RawInstance for closures.

vm/ast.cc

Index: vm/ast.cc
===================================================================
--- vm/ast.cc	(revision 13487)
+++ vm/ast.cc	(working copy)
@@ -287,7 +287,7 @@
 const Instance* ClosureNode::EvalConstExpr() const {
   if (function().IsImplicitStaticClosureFunction()) {
     // Return a value that represents a closure. Only the type is relevant.
-    return &Closure::Handle();
+    return &Instance::Handle();

[Ivan Posva, 2012/10/11 04:04:22]

This does not match with the comment. The type of the handle is now an Instance
and not a Closure. Please either make sure that the type does not matter and
update the comment or we will need to have a closure singleton object so that we
can check for the object being a closure.

[regis, 2012/10/11 15:21:55]

Shouldn't this be a ZoneHandle?

[siva, 2012/10/11 17:45:58]

Changed comment to state that we return a dart instance.

On 2012/10/11 04:04:22, Ivan Posva wrote:

[siva, 2012/10/11 17:45:58]

Checked with Mathias, he thinks it is ok to be a Handle as it is discarded
immediately on return.

On 2012/10/11 15:21:55, regis wrote:

   }
   return NULL;
 }