-
Notifications
You must be signed in to change notification settings - Fork 229
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
ES6 Iterable #65
Comments
Iterable interface for the Enumerable object has been added in this pull request #62, in the sense that you can do But I suppose you're looking for a way to initialize Enumerable from an Iterable object, as described in #51? I think it can be done. I'll see if I can add the feature, that issue kinda slipped below the radar. |
Yes, one solution would be both Iterable and Enumerable to be easily interchangeable. I.e. implement #51 like But I think, better solution would be to Enumerable be Iterable directly? I.e. change |
This is now resolved: 540699b. Can you help verifying the fix? You can now initialize the Enumerable from an iterable object, so the scenario from #51 should now work:
The good part is that it doesn't use an array internally, so it works with infinte iterables...which is pretty sweet.
This will display all even numbers to infinity. |
Yeah, it works perfect, thanks. Just you can use // iterable object
if (Utils.hasNativeIteratorSupport) {
// orig code: if (typeof Symbol !== 'undefined' && typeof obj[Symbol.iterator] !== 'undefined') { |
Cool. Not quite the same, I'll publish to npm later today, thanks for helping out with this. |
Ah, I see, thanks for explanation. |
As a former C# developer I like your library. Just want to know, any plans to use ES6 Iterable instead of custom Enumerable? If you want to support old browsers, would it be possible to have Enumerables to use Iterables under the hood for modern browsers? Also relates to #51
Thanks
The text was updated successfully, but these errors were encountered: