-
Notifications
You must be signed in to change notification settings - Fork 512
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
instanceOf: infinite loop #2780
Comments
@olgierd23 could you provide a jsFiddle reproducing this? |
Unfortunately, I am afraid I could not. It have happened in my system (which is quite big and complicated) some time ago, and I found this issue during debbuging. I fixed it in the above mentiond way and did not make any notes concerning the situation. So now I only know that there was the issue, but can not recover that situation again. :-( |
I am not sure how this matters, you wouldn't cause recursion unless your class somehow extends itself... https://jsfiddle.net/f7qr6zfh/1/ this checks all subclassing examples under mootools and ES6 classes and works correctly. |
How actively it MooTools still being maintained/ developed? |
@stefanklokgieters since ES6 landed, much of what MooTools brought to JavaScript is now Native. Afaik MooTools Discussing this is off-topic here, we had a thread in the Google mailinglist about this. |
In some (rare) cases function instanceOf may fall into an infinite loop, which - in my case - was possible to eliminate by adding additional conditional break.
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: