Comment #2 on issue 582 by [email protected]: Set the right construct
stub for native functions
http://code.google.com/p/v8/issues/detail?id=582
After the fix in
$ ./shell_g --expose-gc
new gc
is OK, however
$ ./shell_g --expose-gc
new gc
new gc
hits the same assert as before. If a for loop is placed in the test test-
api/NativeFunctionConstructCall the assert is also hit.
Index: ../test/cctest/test-api.cc
===================================================================
--- ../test/cctest/test-api.cc (revision 3643)
+++ ../test/cctest/test-api.cc (working copy)
@@ -2839,12 +2839,14 @@
static const char* exts[1] = { "functiontest" };
v8::ExtensionConfiguration config(1, exts);
LocalContext context(&config);
- CHECK_EQ(v8::Integer::New(8),
- Script::Compile(v8_str("(new A()).data"))->Run());
- CHECK_EQ(v8::Integer::New(7),
- Script::Compile(v8_str("(new B()).data"))->Run());
- CHECK_EQ(v8::Integer::New(6),
- Script::Compile(v8_str("(new C()).data"))->Run());
+ for (int i = 0; i < 10; i++) {
+ CHECK_EQ(v8::Integer::New(8),
+ Script::Compile(v8_str("new A().data"))->Run());
+ CHECK_EQ(v8::Integer::New(7),
+ Script::Compile(v8_str("(new B()).data"))->Run());
+ CHECK_EQ(v8::Integer::New(6),
+ Script::Compile(v8_str("(new C()).data"))->Run());
+ }
}
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
