You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If using Janet's stdio repl [1], evaluating the following form leads to on-stray-output being called sometimes:
(do
(print"out")
(error"err"))
IIUC, this occurs because a callback (cb) gets cleared here as a result of on-stdout triggering before on-stderr (and they both call on-message).
By the time on-stderr gets called, cb ends up being bound to on-stray-output because of this.
In my limited testing at the command line (evaluating the same form), stdout output appears to complete after stderr output which is the reverse order of the callbacks being called. I suppose it's possible that that's just the way it could end up being when using event-based things like libuv?
[1] Possibly this may be an issue for other languages too as relevant code lives in fnl/conjure/remote/stdio.fnl.
The text was updated successfully, but these errors were encountered:
for how things are handled in the SQL client. This might work with a Janet REPL.
This is suggested as a simple work-around for handling other stdio clients. When @Olical completes his modularisation work (#500), hopefully, we can improve how the various REPLs work with Conjure.
If using Janet's stdio repl [1], evaluating the following form leads to
on-stray-output
being called sometimes:IIUC, this occurs because a callback (
cb
) gets cleared here as a result ofon-stdout
triggering beforeon-stderr
(and they both callon-message
).By the time
on-stderr
gets called,cb
ends up being bound toon-stray-output
because of this.In my limited testing at the command line (evaluating the same form), stdout output appears to complete after stderr output which is the reverse order of the callbacks being called. I suppose it's possible that that's just the way it could end up being when using event-based things like libuv?
[1] Possibly this may be an issue for other languages too as relevant code lives in fnl/conjure/remote/stdio.fnl.
The text was updated successfully, but these errors were encountered: