Thank, addressed comments. Landing.
https://codereview.chromium.org/1149623010/
--
--
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
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1149623010/20001
https://codereview.chromium.org/1149623010/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are
Patchset 2 (id:??) landed as
https://crrev.com/25981994b781624518540e9fc28c09a40d635245
Cr-Commit-Position: refs/heads/master@{#28818}
https://codereview.chromium.org/1149623010/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this
Committed patchset #2 (id:20001)
https://codereview.chromium.org/1149623010/
--
--
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
lgtm with nits
https://codereview.chromium.org/1149623010/diff/1/src/string-stream.h
File src/string-stream.h (right):
https://codereview.chromium.org/1149623010/diff/1/src/string-stream.h#newcode42
src/string-stream.h:42: ~FixedStringAllocator() {}
override