[Lldb-commits] [clang] [lldb] Fix filename parsing in clang-format-diff.py for paths with spaces (PR #135779)
Selim Keles via lldb-commits
lldb-commits at lists.llvm.org
Thu Apr 17 22:53:01 PDT 2025
https://github.com/selimkeles updated https://github.com/llvm/llvm-project/pull/135779
>From 235ef7b9d0e5f8cb9329400a01fa1b51c74626e7 Mon Sep 17 00:00:00 2001
From: Vy Nguyen <vyng at google.com>
Date: Tue, 15 Apr 2025 11:40:07 +0200
Subject: [PATCH] Fix filename parsing in clang-format-diff.py for paths with
spaces
---
clang/tools/clang-format/clang-format-diff.py | 2 +-
lldb/source/Target/Process.cpp | 9 +++++----
lldb/source/Target/Target.cpp | 2 +-
3 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/clang/tools/clang-format/clang-format-diff.py b/clang/tools/clang-format/clang-format-diff.py
index c82b41e8bd031..3059982ba231f 100755
--- a/clang/tools/clang-format/clang-format-diff.py
+++ b/clang/tools/clang-format/clang-format-diff.py
@@ -102,7 +102,7 @@ def main():
filename = None
lines_by_file = {}
for line in sys.stdin:
- match = re.search(r"^\+\+\+\ (.*?/){%s}(\S*)" % args.p, line)
+ match = re.search(r"^\+\+\+\ (.*?/){%s}(.+)" % args.p, line.rstrip())
if match:
filename = match.group(2)
if filename is None:
diff --git a/lldb/source/Target/Process.cpp b/lldb/source/Target/Process.cpp
index 633f7488dc76a..73557eb767c72 100644
--- a/lldb/source/Target/Process.cpp
+++ b/lldb/source/Target/Process.cpp
@@ -1047,10 +1047,11 @@ bool Process::SetExitStatus(int status, llvm::StringRef exit_string) {
info->exit_desc = {status, exit_string.str()};
});
- helper.DispatchOnExit([&](telemetry::ProcessExitInfo *info) {
- info->module_uuid = module_uuid;
- info->pid = m_pid;
- });
+ helper.DispatchOnExit(
+ [module_uuid, pid = m_pid](telemetry::ProcessExitInfo *info) {
+ info->module_uuid = module_uuid;
+ info->pid = pid;
+ });
m_exit_status = status;
if (!exit_string.empty())
diff --git a/lldb/source/Target/Target.cpp b/lldb/source/Target/Target.cpp
index 42b1561fb2993..b6186b76d6236 100644
--- a/lldb/source/Target/Target.cpp
+++ b/lldb/source/Target/Target.cpp
@@ -1578,7 +1578,7 @@ void Target::SetExecutableModule(ModuleSP &executable_sp,
info->is_start_entry = true;
});
- helper.DispatchOnExit([&](telemetry::ExecutableModuleInfo *info) {
+ helper.DispatchOnExit([&, pid](telemetry::ExecutableModuleInfo *info) {
info->exec_mod = executable_sp;
info->uuid = executable_sp->GetUUID();
info->pid = pid;
More information about the lldb-commits
mailing list