Thanks for quick review!
http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc File test/cctest/test-lockers.cc (right): http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc#newcode616 test/cctest/test-lockers.cc:616: v8::Locker l(isolate); On 2011/06/07 18:28:18, Mads Ager wrote:
l -> lock?
Done. http://codereview.chromium.org/7129002/diff/1/test/cctest/test-lockers.cc#newcode617 test/cctest/test-lockers.cc:617: v8::Isolate::Scope iscope(isolate); On 2011/06/07 18:28:18, Mads Ager wrote:
iscope -> isolate_scope
to match the other naming?
Done. http://codereview.chromium.org/7129002/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
