https://codereview.chromium.org/974213002/diff/120001/src/background-parsing-task.cc
File src/background-parsing-task.cc (left):
https://codereview.chromium.org/974213002/diff/120001/src/background-parsing-task.cc#oldcode20
src/background-parsing-task.cc:20: // We don't set the context to the
CompilationInfo yet, because the background
On 2015/03/09 09:18:07, marja wrote:
This comment is probably still relevant; why was it removed?
Done.
https://codereview.chromium.org/974213002/diff/120001/src/background-parsing-task.cc
File src/background-parsing-task.cc (right):
https://codereview.chromium.org/974213002/diff/120001/src/background-parsing-task.cc#newcode20
src/background-parsing-task.cc:20: Zone* zone = new Zone();
On 2015/03/09 09:18:07, marja wrote:
Who deletes Zone? ParseInfo doesn't own it afaics.
I've changed it back to a SmartPointer in BackgroundParsingTask, so it
will be deleted when the BackgroundParsingTask is.
https://codereview.chromium.org/974213002/diff/120001/src/compiler.cc
File src/compiler.cc (right):
https://codereview.chromium.org/974213002/diff/120001/src/compiler.cc#newcode1099
src/compiler.cc:1099: info->parse_info()->set_cached_data(nullptr);
On 2015/03/09 09:18:07, marja wrote:
Nit: you already have parse_info, so s/info->parse_info()/parse_info/
Done.
https://codereview.chromium.org/974213002/diff/120001/src/compiler.cc#newcode1593
src/compiler.cc:1593: if (parse_info_) {
On 2015/03/09 09:18:07, marja wrote:
Nit: unnecessary if, since delete 0 is ok.
Done.
https://codereview.chromium.org/974213002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
