Skip to content

Commit b2bb0f9

Browse files
authored
[lldb-dap] Minor change to stdio client name. (#141266)
Update the stdin <-> stdout client name to `stdio`. I noticed on macOS if I get a crash report from lldb-dap the `/` in the client name causes some thread names to be reported without the full name. For example `Thread 6 Crashed:: */stdout.event_handler`. Switching to `stdio` I think is still clear and not truncated in the macOS crash reports.
1 parent dc8f2f0 commit b2bb0f9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lldb/tools/lldb-dap/tool/lldb-dap.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,7 @@ int main(int argc, char *argv[]) {
542542
lldb::IOObjectSP output = std::make_shared<NativeFile>(
543543
stdout_fd, File::eOpenOptionWriteOnly, NativeFile::Unowned);
544544

545-
constexpr llvm::StringLiteral client_name = "stdin/stdout";
545+
constexpr llvm::StringLiteral client_name = "stdio";
546546
Transport transport(client_name, log.get(), input, output);
547547
DAP dap(log.get(), default_repl_mode, pre_init_commands, transport);
548548

0 commit comments

Comments
 (0)