[email protected], 2010-09-08 07:29 +0000: > Thank you for the patch.
Thank you for the quick reply. > The fix seems fine, the only thing I would want for it to be complete is a > regression test. The typical place for it would be > tests/mjsunit/regression/regression-857.js (based on the the bug > number). The > other files in that directory show the format. OK, I was expecting to need to provide a regression test anyway, but just wasn't sure what the process was. So, I'll make one and attach it to the issue. > However, before being able to accept a patch, we will need you to sign a > copyright assignment form (see > http://code.google.com/p/v8/wiki/Contributing - > "The small print"). > If you have already signed it, we just need to know what name it's under :) I did already go ahead and complete and submit an individual contributor license agreement earlier today. So, wherever those go after being submitted, there should be one on record for me now, with my [email protected] e-mail address (I'm using my [email protected] address for the bug DB and review DBs just because that's what seems to be required for using those systems.) --Mike > http://codereview.chromium.org/3318017/show -- Michael(tm) Smith http://people.w3.org/mike -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
