Reviewers: titzer,

Description:
Make GCMole happy about Runtime_ForInFoo methods.

[email protected]

Please review this at https://codereview.chromium.org/436523002/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+4, -4 lines):
  M src/runtime.cc


Index: src/runtime.cc
diff --git a/src/runtime.cc b/src/runtime.cc
index 2a0234bd7c05511c11c1e8272a4f8643b9bec489..cb2e8df05d7eea384c6160b52602f2906426383a 100644
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -15143,8 +15143,8 @@ RUNTIME_FUNCTION_RETURN_PAIR(Runtime_ForInInit) {
   // This simulates CONVERT_ARG_HANDLE_CHECKED for calls returning pairs.
   // Not worth creating a macro atm as this function should be removed.
   if (!args[0]->IsJSReceiver() || !args[1]->IsObject()) {
-    return MakePair(isolate->ThrowIllegalOperation(),
-                    isolate->heap()->undefined_value());
+    Object* error = isolate->ThrowIllegalOperation();
+    return MakePair(error, isolate->heap()->undefined_value());
   }
   Handle<JSReceiver> object = args.at<JSReceiver>(0);
   Handle<Object> cache_type = args.at<Object>(1);
@@ -15197,8 +15197,8 @@ RUNTIME_FUNCTION_RETURN_PAIR(Runtime_ForInNext) {
   // Not worth creating a macro atm as this function should be removed.
   if (!args[0]->IsJSReceiver() || !args[1]->IsFixedArray() ||
       !args[2]->IsObject() || !args[3]->IsSmi()) {
-    return MakePair(isolate->ThrowIllegalOperation(),
-                    isolate->heap()->undefined_value());
+    Object* error = isolate->ThrowIllegalOperation();
+    return MakePair(error, isolate->heap()->undefined_value());
   }
   Handle<JSReceiver> object = args.at<JSReceiver>(0);
   Handle<FixedArray> array = args.at<FixedArray>(1);


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to