Memoize simultaneous calls to single call about memoizee HOT 5 CLOSED

@RushPL actually behavior of memoize is as you desire. It won't call async function with same arguments multiple times even if the answer for the initial call has not yet been recieved. All calls would be postponed, and all registered callbacks will be called with result that's obtained once.

If I didn't understand well your issue, or you're sure there's some bug related to that. Please post minimal test case that reproduces it.

I'm going to close it for now (I don't see an issue at this point), but please keep discussion on

from memoizee.

In that case there is a bug. I have observed my function being called two times.

var memoize = require('memoize');

var run = function(str, cb) {
    console.log('run');
    setTimeout(function() {
      cb(null, str + str);
    }, 1000);
};

run = memoize(run, {async: true});

run('ala', function(err, val) {
    console.log('run1');
});

run('ala', function(err, val) {
    console.log('run2');
});

prints:

run
run
run1
run2

clearly a bug

from memoizee.

Are you sure you use memoizee (note the two ee's)? This project is published under this name in npm. You're test case example imports memoize which is different package

from memoizee.

You are right. I wouldn't have guessed. Perhaps rename the repository as well? :) I hope other people don't get discouraged by getting the wrong package.

from memoizee.

Indeed, I think it might be a good idea to rename repository, I'll consider that. Sorry for confusion :)

from memoizee.