LGTM

http://codereview.chromium.org/3130033/diff/5001/6001
File src/utils.h (right):

http://codereview.chromium.org/3130033/diff/5001/6001#newcode743
src/utils.h:743: inline Dest BitCast(Source* & source) {
The linter will probably complaint about this because we are using a
non-const reference.

I don't see why we need the reference at all. I'll change this to just
Source* and submit for you.

Thanks for the patch!

http://codereview.chromium.org/3130033/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to