Replies: 2 comments 14 replies
-
|
Thank you for reporting this. It crashes when the WebUI calls the callback, I'm investigating this. |
Beta Was this translation helpful? Give feedback.
-
As it doesn't fix anything and I can only see disadvantages in this case, I need to ask why this would be better? How I come to say that it isn't better: In the go-binding on a the other hand, having a If such "native-returns" (that infer |
Beta Was this translation helpful? Give feedback.
-
|
As I said earlier, I don't have long experience in V. I thought V would need the V-Event-Handler like the Go and Python wrappers, but based on your explanation, V doesn't need it. So, yeah, we can keep the actual method. About the main issue of this topic, I still need to do more testing to know if it's a C or V issue. |
Beta Was this translation helpful? Give feedback.
-
|
It's hard to isolate the bug. Trying to do so, things became more confusing. Just importing import vwebui as ui
import net.http
const doc = '<html style="background: #654da9; color: #eee">
<head><script src="/webui.js"></script></head>
<body><button onclick="webui.call(\'fetchnews\')">Fetch</button></body>
</html>'
fn main() {
mut w := ui.new_window()
func := fn (_ &ui.Event) {
// Bug happens when clicking the `Fetch` button ~5-10 times.
// Comment out `net.http` and the bug won't happen.
_ := 'a'.repeat(30_000)
/*
// With this fetch, the bug is always happening.
_ := http.get('https://jsonplaceholder.typicode.com/posts') or {
eprintln('Failed fetching news.')
return
}
*/
}
w.bind('fetchnews', func)
w.show(doc)!
ui.wait()
w.destroy()
}The C code V generates can be checked with For now, there definitely must be some conflict that is introduced by combining one of the libs and modules. I assumed it could be related to civetweb. But all I can do for now are blind shots, hoping to hit something that helps narrowing down the problem. With the help of the V community this might get easier to isolate, they have some great people, so I reached out to Vs discord. I'll leave an update if something brings us closer to a solution. |
Beta Was this translation helpful? Give feedback.
-
|
To confirm. import net.http
fn test() {
_ := http.get('https://jsonplaceholder.typicode.com/posts') or {
eprintln('Failed fetching news.')
return
}
}
fn main() {
test()
} |
Beta Was this translation helpful? Give feedback.
-
|
Yep it works fine. It turns out it's not only related to the net.http module. I.e. reading the modfile like below in a vwebui application will also cause this runtime error when having a variable of a certain sizes in a bind cb - like the 'a'.repeat(30_000)` above. A clear relations between the things is yet to be found. import vwebui as ui
import v.vmod
const manifest = vmod.decode(@VMOD_FILE) or { panic(err) }
fn main () {
// or variations like
manifest := vmod.decode(os.read_file('webui/v.mod')!) or { panic(err) }
// ...Compiling with
Edit: |
Beta Was this translation helpful? Give feedback.
-
|
Fixed in webui-dev/v-webui#62 |
Beta Was this translation helpful? Give feedback.
-
I'm encountering this with the V wrapper. To find out if this is an issue that is not limited to V, I thought to start here in the general discussions section first.
Issue:
Using webui and doing a network operation in V - like requesting a page - causes a runtime error resulting in a crash.
Collecting from unknown threadit's an error the garbage collector runs into.The example below, requesting
https://jsonplaceholder.typicode.com/postscauses the error in 10 of 10 tested cases.Requesting a file like
https://hacker-news.firebaseio.com/v0/topstories.jsonwill cause it every now and then. I didn't digged into it deep enough yet, to tell if it's the format or the size that is causing the difference in frequency the error is occurring.Debug LogThe issue is not limited to
net.httpmodule. E.g. if I use a curl binding I'm running into the same issue.I can't get go-webui running atm. @neroist, I hope its not too intrusive to ask if you can test a request like this in nim 😊? When you don't have a free moment I'd check it with nim in the next days as it would be the easiest for me after go and V.
Beta Was this translation helpful? Give feedback.
All reactions