LGTM with comments.
https://chromiumcodereview.appspot.com/9690004/diff/1/ChangeLog File ChangeLog (right): https://chromiumcodereview.appspot.com/9690004/diff/1/ChangeLog#newcode4 ChangeLog:4: (Chromium issue http://code.google.com/p/v8/issues/detail?id=1996) Actually, its a V8 issue :-) "(issue 1996)" would nail it. https://chromiumcodereview.appspot.com/9690004/diff/1/ChangeLog#newcode5 ChangeLog:5: (Patch by [email protected]) Don't need the "patch by" line https://chromiumcodereview.appspot.com/9690004/diff/1/ChangeLog#newcode8 ChangeLog:8: ([email protected]) Don't need this line either. https://chromiumcodereview.appspot.com/9690004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
