http://codereview.chromium.org/10534139/diff/16001/src/compiler.h File src/compiler.h (right):
http://codereview.chromium.org/10534139/diff/16001/src/compiler.h#newcode268 src/compiler.h:268: class ZoneCompilationInfo: public CompilationInfo { On 2012/06/19 20:09:53, danno wrote:
Please call this CompilationInfoWithZone, since both actually create
everything
in a zone, and the difference is that this once has its own zone.
Done. http://codereview.chromium.org/10534139/diff/16001/src/liveedit.cc File src/liveedit.cc (right): http://codereview.chromium.org/10534139/diff/16001/src/liveedit.cc#newcode896 src/liveedit.cc:896: Isolate* isolate = Isolate::Current(); On 2012/06/19 20:09:53, danno wrote:
Why again is it OK to remove the zone scope? Seems necessary...
CompileScriptForTracker creates a CompilationInfoWithZone and hence a Zone and ZoneScope. This wasn't the case before. http://codereview.chromium.org/10534139/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
