On Wed Apr 8 20:07:58 2026 +0000, Jan Robert Gerigk wrote:
I will keep the length check at \> 128 since it matches the CI results. Pushing the update now with a new code quality fix too. And thanks for your patience and Feedback, I only started looking into Wine a few months ago as part of a project to get KNX ETS (a building automation software) running on Linux, and I have only been contributing upstream for a few weeks now. My understanding of Wines internals is still very surface-level, so I really appreciate the feedback. Another push, perhaps this time with no errors!
Updated: length check now rejects strings with 128+ characters (\> 127) to match native behavior. Also fixed the new code quality warning (perhaps). Side note: I notice some Tests in unrelated areas (d3d9, user32, wininet) that sometimes pass, sometimes fail across my pipelines, even though I never touched those Components. Is that expected CI behavior, or should I be concerned? Still new with WineHQ. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10565#note_135597