LGTM. Please update the patch description before submitting.
http://codereview.chromium.org/4116005/diff/6001/7003 File test/mjsunit/string-replace-with-empty.js (right): http://codereview.chromium.org/4116005/diff/6001/7003#newcode67 test/mjsunit/string-replace-with-empty.js:67: // a memory leak. Mention valgrind here as well. http://codereview.chromium.org/4116005/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
