Couple of nits, but LGTM.
https://codereview.chromium.org/653033002/diff/1/src/serialize.cc
File src/serialize.cc (right):
https://codereview.chromium.org/653033002/diff/1/src/serialize.cc#newcode1922
src/serialize.cc:1922: // Allocating a chunk this size won't work any
more. Complete the current
Get rid of "any more," so as to avoid conversations about how it used to
be and currently is in tip of tree. :p
https://codereview.chromium.org/653033002/diff/1/src/serialize.h
File src/serialize.h (right):
https://codereview.chromium.org/653033002/diff/1/src/serialize.h#newcode259
src/serialize.h:259: // We may want to abort and gracefully even if
deserialization fails.
Remove the and to get "...abort gracefully..."
https://codereview.chromium.org/653033002/
--
--
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.
