[Lldb-commits] [PATCH] D151002: [lldb] Fix process pid parsing issue

2023-05-19 Thread Med Ismail Bennani via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGaefa8f4460d1: [lldb] Fix process pid parsing issue (authored by mib). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151002/new/ https://reviews.llvm.org/D15

[Lldb-commits] [PATCH] D151002: [lldb] Fix process pid parsing issue

2023-05-19 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib added a comment. In D151002#4357886 , @bulbazord wrote: > You probably encountered this somewhere, is there a simple test we could add > here? The change looks fine to me nonetheless. This is specific to interactive scripted process and the only tes

[Lldb-commits] [PATCH] D151002: [lldb] Fix process pid parsing issue

2023-05-19 Thread Alex Langford via Phabricator via lldb-commits
bulbazord added a comment. You probably encountered this somewhere, is there a simple test we could add here? The change looks fine to me nonetheless. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151002/new/ https://reviews.llvm.org/D151002

[Lldb-commits] [PATCH] D151002: [lldb] Fix process pid parsing issue

2023-05-19 Thread Med Ismail Bennani via Phabricator via lldb-commits
mib created this revision. mib added reviewers: JDevlieghere, bulbazord. mib added a project: LLDB. Herald added a project: All. mib requested review of this revision. Herald added a subscriber: lldb-commits. This patch should fix an issue when parsing the process pid and setting it in the scripte