aadsm abandoned this revision.
aadsm added a comment.

Fair enough, I haven't seen evidence of this (haven't searched for it) but I 
imagine IDEs need to ignore this as well otherwise they just barf if they're 
expecting `Content-Length` and a wild print appears. The notion of stdout of 
SBDebugger is the SBCommandReturnObject which doesn't print right away (only 
when the command finishes executing) and from my previous tests .flush() 
doesn't help with this. I believe this is the reason why people opt to use 
print instead in their custom commands. But I don't think it's a reasonable 
assumption (or api contract) to tell people they can't use print or it breaks 
lldb-vscode.

I'm going to fix this in lldb-vscode itself to wrap all stdout in a proper DAP 
console response.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70883



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to