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
I noticed that if we replace all queueMicrotask(() => callback( with just callback(, it becomes much much more reliable. So, shall we just do that? Add try catch so that the original calls and setters don't throw.
But why is this happening? Does this mess up the order of operations? Did we maybe forget to use queueMicrotask somewhere, or are using it twice somewhere??
I suspect further issues could be caused by various promise.then()s. Perhaps need to consider having a cached version of things that the promises resolve to. Also this could be related to the queueMicrotask order from the previous point.
Also see TODOs in that file. Some things issues are already known, but it is unknown how often they manifest themselves.
The text was updated successfully, but these errors were encountered:
For example, for this video the
clone
gets the followingclone.error
:I happens immediately as you load the video.
Here's the cloning code: https://github.com/WofWca/jumpcutter/blob/master/src/entry-points/content/cloneMediaSources/lib.ts
queueMicrotask(() => callback(
with justcallback(
, it becomes much much more reliable. So, shall we just do that? Addtry catch
so that the original calls and setters don't throw.But why is this happening? Does this mess up the order of operations? Did we maybe forget to use
queueMicrotask
somewhere, or are using it twice somewhere??promise.then()
s. Perhaps need to consider having a cached version of things that the promises resolve to. Also this could be related to thequeueMicrotask
order from the previous point.The text was updated successfully, but these errors were encountered: