27 Jan
2025
27 Jan
'25
5:58 p.m.
On Mon Jan 27 17:54:09 2025 +0000, William Horvath wrote:
Yes, that was an issue in earlier versions. Your diff makes sense, but we don't have any test cases where `!alertable && queue_apc`, so the `else...if (tests[i].queue_apc)` branch will never run. It should do the work of popping leftover APCs if I just move those outside the `else`.
I think we have, tests 3, 4 counting from 1? If maybe those are not needed then queue_apc should just be removed from the test structure, otherwise it is cleaner to always make sure that there are no APCs queued after each test iteration.