search

speps / XInputDotNet

C# wrapper around XInput, works with any Mono or .NET application (eg. Unity3D)
466 stars 96 forks source link

Check for XInput 1.4 and 9.1.0 when 1.3 is missing #14

Closed hymerman closed 9 years ago

hymerman commented 9 years ago

Self-explanatory! I've tested this on Win 7, Win 8 (with XInput 1.3 installed) and Win 8 (with only XInput 1.4 installed) and XInputReporter.exe now works correctly instead of crashing.

I've not tested Unity yet.

I also don't have any machines with just XInput 9.1.0 so can't test that, but from the docs it looks like it should always exist when XInput 1.4 is installed so I doubt that code will ever get called!

I also snuck in a fix for XInputDemo showing the state of the X button instead of the B button, which @gridgem added in his fork.

hymerman commented 9 years ago

Not sure how I link this to issues, but it will fix #13.

hymerman commented 9 years ago

Oh and I should add, I don't have any machines with VS2010 installed so I tested after upgrading the project to VS2013 - I doubt this will affect the validity of my testing though! Just thought I'd mention it as a disclaimer :)

speps commented 9 years ago

Thanks for that, I'll do a build whenever I can (I should really make that automated...).

hymerman commented 9 years ago

AppVeyor is great for that for Windows projects like this, though it only has VS2013 installed if I remember rightly. Thanks for accepting!