LGTM

http://codereview.chromium.org/1320006/diff/15001/16002
File test/cctest/test-mips.cc (right):

http://codereview.chromium.org/1320006/diff/15001/16002#newcode58
test/cctest/test-mips.cc:58: CHECK_EQ(0x80,
script->Run()->Int32Value());
I was actually thinking of testing both functions, maybe like this:

  const char* c_source_1 = "function foo() { return 0x1234; }; foo();";
  Local<String> source_1 = ::v8::String::New(c_source_1);
  Local<Script> script_1 = ::v8::Script::Compile(source_1);
  CHECK_EQ(0x1234,  script_1->Run()->Int32Value());

And then the other one c_source_2, source_2, script_2 - or make two
scopes.

http://codereview.chromium.org/1320006/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

To unsubscribe, reply using "remove me" as the subject.

Reply via email to