On Fri Mar 27 13:30:55 2026 +0000, Safocl Stollmannovic wrote:
Since the code had (potencially) _undefined behavior_ in this section, it's difficult to predict what was actually intended. I'm really hoping for help from the community and developers in this case. Since it's difficult for me to clearly trace what was supposed to happen here, I would be very grateful if someone could answer the questions I've temporarily posed in the code comments. This is, in any case, fixable within the context of this merge request, and I really appreciate any hints. but if you are talking about converting the code
```cpp uint16 a[2] = {...}; if (*reinterpret_cast<uint32*>(a)){...} ``` into the code ```cpp uint16 a[2] = {...}; const uint16_t hi = a[0]; const uint16_t low = a[1]; if (hi && low){...} ``` , then yes, I consider this change to be logically consistent with the original code. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10471#note_134056