http://bugs.winehq.org/show_bug.cgi?id=6971
ultrageek.lloyd@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ultrageek.lloyd@gmail.com
--- Comment #385 from ultrageek.lloyd@gmail.com 2010-07-19 19:59:44 --- (In reply to comment #351)
Created an attachment (id=29313)
--> (http://bugs.winehq.org/attachment.cgi?id=29313) [details]
Xi2 patch with xorg bug workaround
Here is another patch that works around xorg bug. Try with with. Make sure you do have Xorg 1.8 (aka 7.5).
Tested and confirmed working with Xorg 1.8.1.902 and latest wine git, using Mass Effect 2 as the test case. I was actually just looking into this with the intent to fix it. Guess you beat me to it :P Nice work in any case.
My only gripe with the patch is the profusion of magic numbers. I would suggest using #define's to make the code more readable and more robust, but it looks like that is not common practice in the wine code base. One other thought is that it might be a good idea to translate the rest of the dinput API implimentation to XInput2. It seems like there is already a pretty straightforward 1:1 correspondence between a lot of the dinput functions and XInput2 functions.
Just my 2 cents. Thanks for the patch, and keep up the good work. Now off to play some ME2