LGTM
http://codereview.chromium.org/7864017/diff/4001/src/log.cc File src/log.cc (right): http://codereview.chromium.org/7864017/diff/4001/src/log.cc#newcode1591 src/log.cc:1591: LogExistingFunction(*sfis[i], *code_objects[i]); This is in handle code and it starts by putting its arguments into handles. I think it would be cleaner if it just took its argument as handles. http://codereview.chromium.org/7864017/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
