LGTM, thanks a lot for fixing this Peter. Since this also fixes 1083 we should add a regression test for that as well.
http://codereview.chromium.org/6035014/diff/21001/test/mjsunit/regress/regress-992.js File test/mjsunit/regress/regress-992.js (right): http://codereview.chromium.org/6035014/diff/21001/test/mjsunit/regress/regress-992.js#newcode29 test/mjsunit/regress/regress-992.js:29: // should just update enumerable/configurable flags Period at end of comment http://codereview.chromium.org/6035014/diff/21001/test/mjsunit/regress/regress-992.js#newcode33 test/mjsunit/regress/regress-992.js:33: desc = Object.getOwnPropertyDescriptor(obj, 'p'); var desc = Object.getOwnPropertyDescriptor(obj, 'p'); http://codereview.chromium.org/6035014/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
