jeena/en.javascript.info
1
0
Fork 0
Code Issues Pull requests Projects Packages Wiki Activity Actions

Merge pull request #858 from Senseinvader/patch-1

Browse source 
Update article.md
This commit is contained in:
Ilya Kantor 2019-03-20 16:02:03 +03:00 • committed by GitHub
commit f67d7ad3fe
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
1-js/11-async/06-promisify/article.md
View file

@ -30,7 +30,7 @@ let loadScriptPromise = function(src) {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
loadScript(src, (err, script) => { loadScript(src, (err, script) => {
if (err) reject(err) if (err) reject(err)
elsee resolve(script); else resolve(script);
}); });
}) })
} }
@ -75,7 +75,7 @@ loadScriptPromise(...).then(...);
Here we assume that the original function expects a callback with two arguments `(err, result)`. That's what we meet most often. Then our custom callbacks is exactly in the right format, and `promisify` works great for such case. Here we assume that the original function expects a callback with two arguments `(err, result)`. That's what we meet most often. Then our custom callbacks is exactly in the right format, and `promisify` works great for such case.
But what is the original `f` expects a callback with more arguments `callback(err, res1, res2)`? But what if the original `f` expects a callback with more arguments `callback(err, res1, res2)`?
Here's a modification of `promisify` that returns an array of multiple callback results: Here's a modification of `promisify` that returns an array of multiple callback results: