hippoz/file-manager
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

switch to using microtasks for switching, fixing segfaults

Browse source
This commit is contained in:
hippoz 2022-07-27 02:35:28 +03:00
parent c601beb28e
commit bbec75f451
Signed by: hippoz
GPG key ID: 7C52899193467641
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/DirectoryView.cpp
View file

@ -4,12 +4,15 @@
DirectoryView::~DirectoryView() {} DirectoryView::~DirectoryView() {}
void DirectoryView::update() { void DirectoryView::update() {
window()->start_batch(); // microtasks defer execution of the callback until all external events are processed.
// if they're not used here and update() is called in a click or hover handler,
// chaos will ensue, as the widget tree will be updated during event propagation
window()->queue_microtask([this]() {
target()->clear_children(); target()->clear_children();
for (const auto &entry : std::filesystem::directory_iterator(m_current_path)) { for (const auto &entry : std::filesystem::directory_iterator(m_current_path)) {
auto button = m_target->add<FileButton>(entry.path().filename(), this); auto button = m_target->add<FileButton>(entry.path().filename(), this);
} }
window()->end_batch(); });
} }
void DirectoryView::navigate(std::string path) { void DirectoryView::navigate(std::string path) {