Skip to content

v8 regression invalidates superFunction's primer#10538

Closed
jasonmit wants to merge 1 commit intoemberjs:masterfrom
jasonmit:bad-context
Closed

v8 regression invalidates superFunction's primer#10538
jasonmit wants to merge 1 commit intoemberjs:masterfrom
jasonmit:bad-context

Conversation

@jasonmit
Copy link
Copy Markdown
Member

This takes on the same strategy of https://github.com/jasonmit/ember.js/blob/master/packages/ember-runtime/lib/system/core_object.js#L179-L191

  • Add back args length 2 condition
  • Fix failing recursive super call test

/cc @stefanpenner

Comment thread packages/ember-metal/lib/mixin.js Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we lost the length === 2

@jasonmit
Copy link
Copy Markdown
Member Author

@stefanpenner tests are passing now

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Mar 25, 2015

@stefanpenner - Needs review.

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Aug 2, 2015

@jasonmit - Needs a rebase.

@stefanpenner - Needs review (might want to do a general review before @jasonmit does a rebase)...

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Aug 23, 2015

I'm going to close this because I think the recent work in #12062 and #12132 superceded it.

@rwjblue rwjblue closed this Aug 23, 2015
@jasonmit jasonmit deleted the bad-context branch May 28, 2017 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants