Fixes unsafe multithread mechanism in button_run - Githubissues

hildogjr / KiCost

Build cost spreadsheet for a KiCad project.

MIT License

519 stars 98 forks source link

Fixes unsafe multithread mechanism in button_run #451

Closed set-soft closed 3 years ago

set-soft commented 3 years ago

Fixes #412

On Linux it fixes the error:

[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
python3: ../../src/xcb_io.c:269: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.

The solution is just a copy & paste from StackOverflow: https://stackoverflow.com/questions/32410359/threading-in-wxpython
From what I interpret the main point here is the use of wx.CallAfter

set-soft commented 3 years ago

I changed the base of this to the current HEAD, it should apply cleanly.