http://codereview.chromium.org/7828080/diff/1/src/bootstrapper.cc File src/bootstrapper.cc (right):
http://codereview.chromium.org/7828080/diff/1/src/bootstrapper.cc#newcode1324 src/bootstrapper.cc:1324: INSTALL_NATIVE(JSFunction, "ToPropertyDescriptor", to_property_descriptor); On 2011/09/05 13:48:59, Kevin Millikin wrote:
Do you need this one?
No, removed. http://codereview.chromium.org/7828080/diff/1/src/objects.cc File src/objects.cc (right): http://codereview.chromium.org/7828080/diff/1/src/objects.cc#newcode2313 src/objects.cc:2313: Handle<String> enum_name = isolate->factory()->LookupAsciiSymbol("enumerable"); On 2011/09/05 13:48:59, Kevin Millikin wrote:
It looks like there are a couple of long lines. Check presubmit
before
committing.
Done. http://codereview.chromium.org/7828080/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
