diff options
| author | Pavel Labath <pavel@labath.sk> | 2025-06-25 08:09:36 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-06-25 08:09:36 +0200 |
| commit | 0512d119fdf019b7c56e58f89b094ee5928b0a07 (patch) | |
| tree | bacc7e9909d8420e57bb7c472c6a0a6ec1481bc9 /lldb/packages/Python/lldbsuite/test/lldbpexpect.py | |
| parent | 888f84f72c07dfad8f266d907799c4060835a960 (diff) | |
| download | llvm-0512d119fdf019b7c56e58f89b094ee5928b0a07.zip llvm-0512d119fdf019b7c56e58f89b094ee5928b0a07.tar.gz llvm-0512d119fdf019b7c56e58f89b094ee5928b0a07.tar.bz2 | |
[lldb] Clean up GDBRemoteCommunication::StartDebugserverProcess (#145021)
The function was extremely messy in that it, depending on the set of
arguments, it could either modify the Connection object in `this` or
not. It had a lot of arguments, with each call site passing a different
combination of null values. This PR:
- packs "url" and "comm_fd" arguments into a variant as they are
mutually exclusive
- removes the (surprising) "null url *and* null comm_fd" code path which
is not used as of https://github.com/llvm/llvm-project/pull/145017
- marks the function as `static` to make it clear it (now) does not
operate on the `this` object.
Depends on #145017
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/lldbpexpect.py')
0 files changed, 0 insertions, 0 deletions