[l for (e in yield)]
asserts js debug shell on TM changeset 7dee6c83aaa2 without any CLI arguments at Assertion failure: fp->isGeneratorFrame(), and crashes js opt shell at JSContext::generatorFor
Reproduces after the fix for bug 665286 landed on TM.
Created attachment 541687 [details] [diff] [review]
added missing call to maybeNoteGenerator()
Glah, missed a case where I needed to call maybeNoteGenerator(). There is now one maybeNoteGenerator() call for every GenexpGuard, which is as it should be.
Laziness + mutation, so happy together. :}
cdleary-bot mozilla-central merge info:
Testcases have been landed by virtue of being marked in-testsuite+ -> VERIFIED as well.