Thanks a lot, Kasper! Submitting.
http://codereview.chromium.org/155635/diff/2001/15 File src/api.cc (right): http://codereview.chromium.org/155635/diff/2001/15#newcode2458 Line 2458: // Fast case, aligned native pointer On 2009/07/17 10:01:39, Kasper Lund wrote: > Terminate comment with . Done. http://codereview.chromium.org/155635/diff/2001/15#newcode2464 Line 2464: // Play safe even if it's something unexpected On 2009/07/17 10:01:39, Kasper Lund wrote: > Terminate comment with . Done. http://codereview.chromium.org/155635/diff/2001/15#newcode2469 Line 2469: // Unaligned native pointer On 2009/07/17 10:01:39, Kasper Lund wrote: > Terminate comment with . Done. http://codereview.chromium.org/155635/diff/2001/15#newcode2490 Line 2490: if (reinterpret_cast<i::Object*>(value)->IsSmi()) { On 2009/07/17 10:01:39, Kasper Lund wrote: > Introduce a local variable to avoid reinterpret_cast'ing the value to i::Object* > twice. Done. http://codereview.chromium.org/155635/diff/2001/15#newcode2494 Line 2494: // Currently internal fields used by DOM wrappers which On 2009/07/17 10:01:39, Kasper Lund wrote: > used -> are used > which got GCed by mark phases -> which only get GCed by the mark-sweep collector Done. http://codereview.chromium.org/155635 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
