-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Panic in ListBox.ConnectRowSelected #130
Comments
Could you try a1bfe15? |
No success unfortunately, but it's a different warning:
With this one, I'm also getting crashes at startup I've never seen before (approximately every fifth start): segfault
concurrent map read and map write
|
Yeah, there's a lot that needs to be worked on here. Unfortunately, my computer is really struggling, so it might take a while for me to debug this. In the meantime, work is being done at the exp-130 branch. |
The M1 CPU saves the day again... |
There is no way to cache the cgo compilation is there? A full rebuild takes like 10 minutes on my 5th gen Ryzen machine 😴 |
Unfortunately not in any way that's useful in this case, since I'm modifying package |
Could you try 2cc5276? |
Nope
I'll see if I can put together a minimal reproduction project later |
Sorry, could you try fc1a77f? |
To avoid cluttering the comments too much, there's also now a Matrix room if you prefer using that instead. The link is in the README now. |
I've noticed I now get a panic when closing and reopening a window containing a list, which has never happened before the memory leak fix #126
It's reproducible in my app:
When I comment the RowSelected signal handler, it goes away https://github.com/getseabird/seabird/blob/195ce324f2cc0c15570544c53d4aadb88fbc8a8b/ui/navigation.go#L107
Tried to make a minimal reproduction and couldn't...if you need more info, I'll try again
The text was updated successfully, but these errors were encountered: