[Lldb-commits] [PATCH] D35298: [MainLoop] Fix possible use of an invalid iterator

2017-07-12 Thread Phabricator via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL307782: [MainLoop] Fix possible use of an invalid iterator (authored by petr.pavlu). Changed prior to commit: https://reviews.llvm.org/D35298?vs=106164=106187#toc Repository: rL LLVM

[Lldb-commits] [PATCH] D35298: [MainLoop] Fix possible use of an invalid iterator

2017-07-12 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. Looks great, thanks for catching this. The code was doing it this way initially, but then this got lost with all the frantic refactors. https://reviews.llvm.org/D35298

[Lldb-commits] [PATCH] D35298: [MainLoop] Fix possible use of an invalid iterator

2017-07-12 Thread Petr Pavlu via Phabricator via lldb-commits
petpav01 created this revision. Store file descriptors from `loop.m_read_fds` (if `FORCE_PSELECT` is defined) and signals from `loop.m_signals` that need to be processed in `MainLoop::RunImpl::ProcessEvents()` into a separate vector and then iterate over this container to invoke the callbacks.