Commit 5628fcf
authored
## Purpose
Fix the implementation of `QSaveRegisterState` and
`QRestoreRegisterState` to work even with no thread identifier, per the
[lldb packet
documentation](https://lldb.llvm.org/resources/lldbgdbremote.html#qrestoreregisterstate-save-id-qrestoreregisterstate-save-id-thread-xxxx).
## Overview
* Fix handling of `QSaveRegisterState` to use the current thread if none
was specified in the packet arguments.
* Fix handling of `QSRestoreRegisterState` to use the current thread if
none was specified in the packet arguments.
## Validation
With PR #143 applied, the
`TestGdbRemoteRegisterState.test_grp_register_save_restore_works_no_suffix_llgs`
passes locally running against DS2 on Linux x86_64.
1 parent 21ba9e3 commit 5628fcf
1 file changed
+12
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
672 | 672 | | |
673 | 673 | | |
674 | 674 | | |
675 | | - | |
| 675 | + | |
| 676 | + | |
| 677 | + | |
| 678 | + | |
| 679 | + | |
| 680 | + | |
676 | 681 | | |
677 | 682 | | |
678 | 683 | | |
| |||
687 | 692 | | |
688 | 693 | | |
689 | 694 | | |
690 | | - | |
| 695 | + | |
| 696 | + | |
| 697 | + | |
| 698 | + | |
| 699 | + | |
| 700 | + | |
691 | 701 | | |
692 | 702 | | |
693 | 703 | | |
| |||
0 commit comments