Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

bump spidermonkey #356

Merged
merged 1 commit into from
Nov 26, 2015
Merged

bump spidermonkey #356

merged 1 commit into from
Nov 26, 2015

Conversation

magopian
Copy link
Contributor

We need a version greater than 43.0a2 for bug 1224444

'var foo = function () { %s; }',
'var foo = function () { %s; yield 1; }',
'var foo = function* () { %s; yield 1; }',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those changes are needed for the tests to pass with this version of spidermonkey. Otherwise they raise a syntax error instead.

According to https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/Legacy_generator_function, "The legacy generator function is a SpiderMonkey-specific feature, and will be removed at some point." (thanks @muffinresearch for digging that up).

@magopian
Copy link
Contributor Author

@kmaglione would you mind giving this a quick r? please? I'm worried about the syntax change, should we simply go with it?

I've looked through some results on https://mxr.mozilla.org/addons/search?string=yield&find=&findi=&filter=&hitlimit=&tree=addons and found many add-ons using the (now deprecated?) function for generators. If we do upgrade spidermonkey, then we'll break the validation for all of those, right?

@magopian magopian force-pushed the bump-spidermonkey branch 2 times, most recently from d837e01 to a9383a3 Compare November 19, 2015 10:29
@magopian
Copy link
Contributor Author

As a stop-gap, i've added a version(180) to force the version of js (to not be 185, which deprecates the function for generators, see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/Legacy_generator_function)

@EnTeQuAk
Copy link
Contributor

r+

magopian added a commit that referenced this pull request Nov 26, 2015
@magopian magopian merged commit 8fbd554 into mozilla:master Nov 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants