Open hazyone opened 1 year ago
Also, as I understood, when I'm applying patch, I get +1 cetus instance in the ui picker(multi wasm mode), but at this moment, in the consolecetusInstance
contains only 1 active instance
cetusInstances._instances.length
1
Also, I can't search for the '0' i32 value, got error:
cetus.js:230 Uncaught RangeError: Invalid array length
at Cetus._compare (cetus.js:230:47)
at Cetus.search (cetus.js:426:34)
at cetus.js:902:38
this._searchSubset[i] = currentValue; << here
Hey @hazyone
Good finds here! I noticed there's a bunch of regressions relating to the new "multi-binary" support so I finally took the time to implement automatic regression testing. Hopefully this means we won't run into cases like this as much in the future.
In the meantime, I fixed a few of these issues and will continue working on the rest in the morning.
Jack
Thank you, you're doing a great job. I like your tool :)
I think these issues are fixed, except for the searching for 0 bug. This occurs because searching for the value 0 returns so many results that the browser isn't able to construct a large enough array. I should do some better error handling in this case, but it wouldn't really solve the problem of being unable to search for 0.
In general, I try to avoid searching for common values (Like 0) unless I've already narrowed it down to a small memory space. Otherwise, it just becomes too many search results for the browser (or my brain) to handle.
Jack
Hey! :) I've found, that I can't use
watch
for my bookmarks and also I can't delete them on the latestmaster
.I've got an error:
I've tried to fix this error:
And the error disappeared, but this not fixed
watch
functions