Tomas Carnecky wrote:
So.. now that the cause is known, what would be the right solution? Do you still think it's the WMs fault? Or should wine be changed?
The patch in the attachment removes update_key_state() from mouse.c - the decision whether this is the right approach or not is left to someone else. If you think it's worth a try to send this patch to wine-patches, please do so, I'll have to continue using a custom patchset anyway since the app I want to run doesn't work on vanilla wine, so I don't heave really a strong interest in getting this patch accepted.
Getting patches accepted was quite hard for me, and that was because lack of feedback. It leads to nowhere if I submit the very same patch more than twice, and without feedback I can't improve it. So, my two patches (one for alsa/dsound and this for the stuck keys) will remain public domain and free for everyone to use, and if you feel like it, to improve and send back to wine for inclusion.
I'll (more or less) retire from writing patches. But of course if I come up with a fix for something that I think is worth sharing, I'll let you guys know ;)
tom