Ark/leporello-js
1
0
Fork 0
mirror of https://github.com/leporello-js/leporello-js synced 2026-01-13 21:14:28 -08:00
Code Issues Packages Projects Releases Wiki Activity

do not save promise status when promise returned from fn, only on then and await

Browse Source
This commit is contained in:
Dmitry Vasilev
2022-12-25 18:20:47 +08:00
parent 5b8a6526f6
commit 90ee19c342
1 changed files with 8 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/eval.js
View File

@@ -395,24 +395,6 @@ export const eval_modules = (
} }
} }
const set_promise_status = value => {
if(value instanceof Promise) {
// record stack for async calls, so expand calls works sync
set_record_call()
// TODO why we set status for wrapped value and not for wrapper?
return value
.then(v => {
value.status = {ok: true, value: v}
return v
})
.catch(e => {
value.status = {ok: false, error: e}
throw e
})
} else {
return value
}
}
const __do_await = async value => { const __do_await = async value => {
// children is an array of child calls for current function call. But it // children is an array of child calls for current function call. But it
@@ -473,7 +455,10 @@ export const eval_modules = (
try { try {
value = fn(...args) value = fn(...args)
ok = true ok = true
return set_promise_status(value) if(value instanceof Promise) {
set_record_call()
}
return value
} catch(_error) { } catch(_error) {
ok = false ok = false
error = _error error = _error
@@ -573,7 +558,10 @@ export const eval_modules = (
value = undefined value = undefined
} }
ok = true ok = true
return set_promise_status(value) if(value instanceof Promise) {
set_record_call()
}
return value
} catch(_error) { } catch(_error) {
ok = false ok = false
error = _error error = _error