Attempting to use forEach for consistency

jhiesey / stream-http

Streaming node http in the browser

MIT License

354 stars 62 forks source link

Closed neerjoshtwitch closed 8 years ago

neerjoshtwitch commented 8 years ago

Recent MS Windows 10 anniversary updates, seems to have fetch implementations. However, iterators are not supported (none listed on their dev page )

This change is to rely on forEach instead of iterators.

neerjoshtwitch commented 8 years ago

@jhiesey Please review

kevva commented 8 years ago

@neerjoshtwitch, why was this and #49 closed?

gillstrom commented 8 years ago

This PR seems to pass all tests. Also tested with browserify in MS Edge 38, Chrome, IE 11 and Firefox.

As @kevva said: what is the reason for closing this?