-
Notifications
You must be signed in to change notification settings - Fork 2
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
Why not Lo-dash? #5
Comments
Well, at first glance, underscore or lo-dash are very similar, You would really have to know the very specifics of the implementation as to why the one would be better than the other (better performance, more interesting API ...); which is something that I have never investigated. If there are fixes possible in underscore, based on lo-dash (or the other way around), then I think that it is rather something between them to discuss, unless it they do not agree, or possibly do not even like each other (which quite often happens in fork situations). So, yes, it is certainly possible that lo-dash has better performance than underscore (or the other way around), but I have actually never tried to investigate that. If I remember it right, they are quite API compatible, aren't they? |
Well, here it says:
so if this is true then replacing underscore with Lo-Dash in eventemitter4 should be as easy as - _=require('underscore');
+ _=require('lodash'); |
As to what performance impact such replacement would make, I guess it depends largely on which / how eventemitter4 uses underscore functions. This would have to be tested :) |
Wouldn't performance of eventemitter4 improve by replacing underscore with the (generally more performant) lo-dash library? Or does it rely on some specific underscore functions that lo-dash doesn't offer?
The text was updated successfully, but these errors were encountered: