Re: [PATCH v6 0/2] MR8360: include: Don't evaluate format arguments to ok() unless we need them.

[Bernhard Übelacker]

Hello @yshui, I am sorry, but I think I found an unwanted side effect of this patch.

Since a9edbf49 got merged a failure in the [test pattern page of advapi32:registry](https://test.winehq.org/data/patterns.html#advapi32:registry) showed up.

```
registry.c:2267: Test failed: RegQueryValueExA returned: 0
```

Because I was not able to reproduce it with testbot or a VM I tried to use gitlab CI:
- [failure shows up here with a9edbf49](https://gitlab.winehq.org/bernhardu/wine/-/jobs/206300#L82)
- [but succeeds here without a9edbf49](https://gitlab.winehq.org/bernhardu/wine/-/jobs/206303#L82)
- Unfortunately adding a few traces around the place makes the failure go away like [here](https://gitlab.winehq.org/bernhardu/wine/-/jobs/209460#L82)
- Also disabling optimizations for the function containing the test did not help like [here](https://gitlab.winehq.org/bernhardu/wine/-/jobs/209463#L83)

(The git history of these Jobs can still be accessed by following the "Commit" at the right side, button "Browse Files", button "History.)

What do you think?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8360#note_122996