Almost there, just three more nits.
https://chromiumcodereview.appspot.com/10411067/diff/7001/src/objects.cc File src/objects.cc (right): https://chromiumcodereview.appspot.com/10411067/diff/7001/src/objects.cc#newcode4922 src/objects.cc:4922: ((raw_index & 1) == 0) ? accessors->getter():accessors->setter(); Spaces to the left and right of ":". I know it will break the line, but we just don't compress operators to avoid line breaking. https://chromiumcodereview.appspot.com/10411067/diff/7001/src/objects.h File src/objects.h (right): https://chromiumcodereview.appspot.com/10411067/diff/7001/src/objects.h#newcode2423 src/objects.h:2423: friend class IntrusiveMapTransitionIterator; Move that to the bottom of the class (or at least into the private section). https://chromiumcodereview.appspot.com/10411067/diff/7001/src/objects.h#newcode2480 src/objects.h:2480: Remove this empty line. https://chromiumcodereview.appspot.com/10411067/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
