On 2015/09/08 07:10:34, cbruni wrote:
Doesn't merge right now, I think due to some merge artifacts (I added two
comments).
Right, I am going to rebase the patch, upload a CL with the rebase.
Then I'll go through the suggested changes and upload a new CL just
with that, to make it easier to review.
https://codereview.chromium.org/1309243003/diff/1/src/debug/liveedit.js
File src/debug/liveedit.js (right):
https://codereview.chromium.org/1309243003/diff/1/src/debug/liveedit.js#newcode219
src/debug/liveedit.js:219: for (var j = 0; j <
update_positions_list[i].live_shared_function_infos.length; j++) {
Did you change this on purpose? Seems unrelated to your fix.
This was on purpose. Not doing it would make the bootstrapper raise
an exception and mksnapshot would not finish correctly. I'see if we
can avoid the change once the code is uses a macro like you suggest.
https://codereview.chromium.org/1309243003/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
