return value on set by jfromaniello · Pull Request #12 · dominictarr/hashlru

jfromaniello · 2017-03-22T16:32:07Z

This is handy because you can avoid an if statement, instead of doing

var value = lru.get('foo');
if (!value) {
 value = compute('foo');
 lru.set('foo', value);
}
...

You can do:

const value = lru.get('foo') || lru.set('foo', compute('foo'));
...

Some of the other LRU implementations on npm I have check implement this.

davidmarkclements · 2017-03-22T20:56:00Z

just to be absolutely sure - can we just make sure it doesn't affect the benchmarks in any significant way?

can you please run benchmarks for master and branch and post results?

return value on set

2309923

Provide feedback