Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

named function expression #8

Closed
tbranyen opened this issue Jun 9, 2010 · 8 comments
Closed

named function expression #8

tbranyen opened this issue Jun 9, 2010 · 8 comments

Comments

@tbranyen
Copy link

tbranyen commented Jun 9, 2010

In self.start there is a named function expression: loopy which will cause issues in IE browsers, but more specifically the blackberry browser. One possible solution is to remove the function name and use arguments.callee... although I'm not sure if this is kosher.

@cowboy
Copy link
Owner

cowboy commented Jun 12, 2010

Lark, afaik in IE, the NFE name might leak out of scope, but shouldn't have any other issues. And of course, it doesn't need to be an NFE (I just happen to like that pattern).

What do these 3 lines output for you in the Blackberry browser?

(function x(){ alert( typeof x ); })();
var y; (y = function(){ alert( typeof y ); })();
function z(){ alert( typeof z ); }; z();

@tbranyen
Copy link
Author

tbranyen commented Jul 1, 2010

undefined, function, function

@cowboy
Copy link
Owner

cowboy commented Jul 5, 2010

This bug appears to have been fixed as a side-effect of c164cce, so I'm closing it!

@kangax
Copy link

kangax commented Dec 17, 2010

@lark Which version of a Blackberry is it?

@tbranyen
Copy link
Author

tbranyen commented Jan 3, 2011

@kangax 4.6

@kangax
Copy link

kangax commented Jan 3, 2011

@lark Hm, this is the latest one, isn't it (do you know which model)? It would be pretty sucky if latest blackberry isn't able to create NFE properly.

@tbranyen
Copy link
Author

tbranyen commented Jan 3, 2011

Nope, latest is 6. 5 came in between that and doesn't have this issue. 4.6 is like the IE 6 of mobile web, so its hardly relevant. Even jQuery mobile is ignoring it due to currentStyle reflow issues.

@kangax
Copy link

kangax commented Jan 3, 2011

Gotcha.

I remember having other NFE "issues" in Blackberry but unfortunately haven't documented which version it was :/ http://kangax.github.com/nfe/#spidermonkey-peculiarity

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants