fent
fent
hey @zhimsel i no longer use this plugin, so i'm not as incentivized to go through with the change. plus, i'm not a huge fan of adding options to fix...
The callback should be the last argument in the function. This is standard node async function design.
You could make `skip_status` argument optional. ``` js if (typeof skip_status === 'function') { callback = skip_status; skip_status = false; } ```
I made this change in pr #72 But this pr is more focused on that specific issue.
This looks like it's really simple to do. Just replace `EventEmitter` with `Stream` here https://github.com/AvianFlu/ntwitter/blob/master/lib/parser.js#L13 and return the `stream` object instead of the ntwitter instance here https://github.com/AvianFlu/ntwitter/blob/master/lib/twitter.js#L284 And then add...
I created a module for parsing coutinuous json strings https://github.com/fent/node-jstream Might be faster for this since data can be streamed into the parser. It uses [clarinet](https://github.com/dscape/clarinet)
for youtube it would be simple, youtube will often specify below the description if the video is a song. but, it will be different for every site, introducing this feature...
Makes more sense. Sounds good to me.