25 Jun
2025
25 Jun
'25
1:15 p.m.
On Wed Jun 25 13:15:16 2025 +0000, Elizabeth Figura wrote:
I believe the intent of that check *was*, simply, "mode_idx < mode_count". We can just change the ok() to that. We don't need an "if (...)" continue afterward, either. A test failure is a test failure. the `if()` is needed because there are uses of `modes[mode_idx]` later, which would be out of bound.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8421#note_107899