You're correct that debuggers can be sensitive to the timing of the initial breakpoint.
Adding a conformance test would let us confirm your observation, and ensure such regression won't happen in the future. For reference, see 2b40969c58a099f958c58909b666915991a42979.
I'm still approving since the existing tests did not fail and I cannot find a particular reason `process_breakpoint();` must be placed exactly there in the commit history.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4655#note_56412
DMUS_PMSG::dwType is the type of the message, but we are comparing it to
curve types. We should be using DMUS_CURVE_PMSG::bType instead.
* * *
(I tried to write some test cases for this. I created a IDirectMusicPort implementation that checks the generated midi commands, but instead I cannot get Windows to generate any midi commands for CCCURVEs. )
--
v2: dmime: Fix handling of curve PMSG.
https://gitlab.winehq.org/wine/wine/-/merge_requests/4703
A temporary variable is used here to assist with assignment
but this does not conform to the coding style in the rest of
gdiplus and introduces an unnecessary variable.
Signed-off-by: David Kahurani <k.kahurani(a)gmail.com>
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4734
Currently there is `NtContinue()` test only for `aarch64` so implement it for amd64 aswell.
This implementation is very similar to `aarch64` and it's very basic test.
It doesn't change/test `ContextFlags` so it won't catch https://bugs.winehq.org/show_bug.cgi?id=56050 but that can be implemented later on top of this.
--
v2: ntdll/tests: Implement test_continue() for amd64
https://gitlab.winehq.org/wine/wine/-/merge_requests/4720