Thanks for the patch. I'll take care of these comments and land.
http://codereview.chromium.org/6759029/diff/1/src/execution.cc File src/execution.cc (right): http://codereview.chromium.org/6759029/diff/1/src/execution.cc#newcode556 src/execution.cc:556: Isolate* isolate = Isolate::Current(); Get the isolate from the string instead of going through TLS? http://codereview.chromium.org/6759029/diff/1/src/execution.cc#newcode588 src/execution.cc:588: Isolate* isolate = Isolate::Current(); Get the isolate from the data instead of going through TLS? http://codereview.chromium.org/6759029/diff/1/src/execution.cc#newcode607 src/execution.cc:607: Isolate* isolate = Isolate::Current(); Ditto. http://codereview.chromium.org/6759029/diff/1/src/execution.cc#newcode759 src/execution.cc:759: debug_command_only); Indentation is off. http://codereview.chromium.org/6759029/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
