nyamatongwe left a comment (geany/geany#4620)

Here is a patch that avoids `std::async` and `std::future` for the single 
threaded case.

[RunThreads.patch.txt](https://github.com/user-attachments/files/30245326/RunThreads.patch.txt)

> Alternatively, @nyamatongwe found a new bug in that Geany isn't using 
> SCI_SETLAYOUTTHREADS.

Multi-threading layout isn't a significant improvement for reasonable sized 
files, only for huge files.

> replaced it with a fob off to gio ... Let me clean this up a little and I'll 
> get a pull opened.

The code in `EditView.cxx` and `Editor.cxx` is cross-platform so runs on Win32, 
macOS, and Qt (and other out-of-tree platforms) without gio. It would not be 
reasonable to add gio-specific threading to these files.



-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/4620#issuecomment-5039824089
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/issues/4620/[email protected]>

Reply via email to