Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed a bug with IE8 blowing up when .apply is called with the explicit second parameter set to undefined #17

Merged
merged 1 commit into from Jul 13, 2011

Conversation

Projects
None yet
2 participants
Contributor

valueof commented Jul 12, 2011

The code below works in all modern browsers, including IE9, but blows up in IE8.

var args = undefined;
function fn() {}
fn.apply({}, args);
> Object expected

My patch fixes that in a couple of places so that bean's tests pass.

fat added a commit that referenced this pull request Jul 13, 2011

Merge pull request #17 from antonkovalyov/master
Fixed a bug with IE8 blowing up when .apply is called with the explicit second parameter set to undefined

@fat fat merged commit af12f3e into fat:master Jul 13, 2011

Owner

fat commented Jul 13, 2011

Awesome! nice catch :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment