On Fri May 2 12:03:34 2025 +0000, Connor McAdams wrote:
changed this line in [version 3 of the diff](/wine/wine/-/merge_requests/7711/diffs?diff_id=175186&start_sha=f53783959ae8bd44553a5d89031bc043eeb67e38#d49071753fdf2ab21d518dbd8c720b48c7c732fc_2286_2429)
This ended up changed back to pixel-by-pixel comparisons now due to adding new tests for lower bpc formats, our conversion to 8bpc doesn't exactly match native.
It was quite a few MRs ago, but at one point I had left code similar to this, and the idea was that it was only necessary to count while there were still existing `todo` tests. Once all tests were expected to pass, it made sense to get rid of the extra `mismatch_count` variable and just test each line directly. I've avoided doing that in the current revision and left `mismatch_count` in, even after all tests are expected to pass.