[Lldb-commits] [PATCH] D80659: [lldb-vscode] Redirect stderr and stdout to DAPs console message

[Richard Howell via Phabricator via lldb-commits]

rmaz added a comment.



In D80659#2062304 <https://reviews.llvm.org/D80659#2062304>, @labath wrote:

> Btw, given that Richard has found some vscode code which already tries to send stderr to the console window https://github.com/microsoft/vscode/blob/master/src/vs/workbench/contrib/debug/node/debugAdapter.ts#L234, it would be good to first understand why doesn't that kick in. Maybe there's a vscode bug that could be fixed instead.


The issue is fairly clear, that listener is only added if the optional constructor arg `outputService` is defined, but it never is. I was planning on putting up a diff this week some time to always read from stderr regardless.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80659