On Wed Apr 8 19:41:00 2026 +0000, Jan Robert Gerigk wrote:
The documentation says MAX_LENGTH = 130 with "space included for NULL terminator", which I initially read as 129 chars + NULL = 130. But native rejects 129 and accepts 128. Is the correct interpretation that the constant includes additional padding beyond the NULL terminator (128 chars + NULL + 1 reserved = 130)? Or is the documentation simply off by one? I am really not sure on this point, I have no idea how to read that right. Anyway, back to 128? Well.. we definitely want the constants in the header to match Windows... Honestly maybe it's just not worth worrying about the length check. It's a very unlikely scenario that it would matter anyway; it's not like that string is going to be dynamically generated. Sorry this turned out to be so complicated; I assumed it'd just be a simple check.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10565#note_135553