[Lldb-commits] [lldb] [lldb] Adding file and pipe support to lldb_private::MainLoopWindows. (PR #145621)

[John Harrison via lldb-commits]

ashgti wrote:

> * do we need regular file support

We don't strictly need this for anything I am aware of, I can remove it for now. A 'better' solution would be to stat the file and then watch the directory for changes and re-stat the file. Thats used in libuv (nodejs). I think libdispatch on Windows doesn't actually do anything for files, so you don't get notified if a file on disk changes.

> * do we need the pipe _thread_

In order to support both anonymous pipes, like stdin, and named pipes we would need the thread.

We could specialize this further and have the thread only for anonymous pipes and use a different strategy for named pipes, but the current solution works for both.


https://github.com/llvm/llvm-project/pull/145621