LGTM.

http://codereview.chromium.org/115080/diff/1/2
File src/serialize.cc (right):

http://codereview.chromium.org/115080/diff/1/2#newcode733
Line 733: return entry == NULL ? -1 :
I would put ? and : on separate lines like this:

   return entry == NULL
       ? -1
       : ....;

http://codereview.chromium.org/115080

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

Reply via email to