Reviewers: Mads Ager, plesner, Erik Corry, Message: Minor changes to the API and some code movement to enable an internal HandleScope. Please verify that the API addition makes sense.
And guess what? The verification already turned up at least one missing v8::Locker in d8.cc. I am looking at fixing the -p support in d8.cc, I will roll the fix for the missing locker in with that. -Ivan Description: Split handle scopes into an internal version and a version accessible through the API. This allows us to verify state on entry through the API. In this change verification in the API entry is checking that the current thread holds the V8 lock when a HandleScope is instantiated if a v8::Locker has ever been used by the V8 instance. Please review this at http://codereview.chromium.org/18707 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M include/v8.h M src/api.h M src/api.cc M src/factory.cc M src/handles-inl.h M src/handles.h M src/handles.cc M src/messages.cc M src/v8threads.cc M tools/v8.xcodeproj/project.pbxproj --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
