| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
potenitally fixes node-cache#215, needs more testing though note: I have no idea of coffeescript, is my fix syntacitally correct?
|
Oh right, some test fail now due to the async behaviour. Either we await the setImmediate callback (Can we use async / await in coffeescript?) or we change some other code logic here to let this thing happen asynchronously. |
Sorry, something went wrong.
|
Javascript code that solves the issue on my side: async _checkData(startPeriod = true) {
var key, ref, value;
boundMethodCheck(this, NodeCache);
ref = this.data;
// run the housekeeping method
var i = 0;
for (key in ref) {
value = ref[key];
this._check(key, value);
if (i % 10 === 0) {
await new Promise(resolve => setImmediate(resolve));
}
i++;
}
.... |
Sorry, something went wrong.
this potentially breaks support for older node versions, sorry, I have no idea about coffeescript, just testing out ;)
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
potenitally fixes #215, needs more testing though
note: I have no idea of coffeescript, is my fix syntacitally correct?