[Lldb-commits] [PATCH] D93951: [vscode] Improve runInTerminal and support linux

[walter erquinigo via Phabricator via lldb-commits]

wallace added a comment.

I'll follow your recommendations.

Regarding Windows, mkfifo can be replaced with CreateNamedPipe and should lead to the same behavior. Sadly, I don't have a Windows machine where I could even compile it... So this should be left for someone else or for when I can get my laptop from the office.

And why are fifo files needed? Well, when you open a normal text file, c++ doesn't just know when the file gets updated. Its pointer to the end of the file is set when you open the file. So if you want to get updates, you have to reopen the file in a polling fashion. Fifo files, on the other hand, which are the same as pipes, behave the same way as stdin, so c++ keeps reading it until it's empty and closed at OS level, aware of new data being written to it


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93951/new/

https://reviews.llvm.org/D93951