aardappel / procrastitracker

a Windows time tracking application
http://strlen.com/procrastitracker/
510 stars 54 forks source link

Xinput activity detection #45

Closed nuzayets closed 6 years ago

nuzayets commented 6 years ago

Monitor Xinput Controllers for Activity

aardappel commented 6 years ago

This generally looks good to me, though I haven't tested if it works. Let me know if you think it is ready to merge.

nuzayets commented 6 years ago

I will let you know after I run it for a few days of real use. Thanks!

nuzayets commented 6 years ago

It seems to work well. I haven't been able to trip it up. Ran it for multiple days without a false positive too.

aardappel commented 6 years ago

Ok, I'll merge! Thanks for doing this :)

aardappel commented 5 years ago

@nuzayets FYI: The dependency on Xinput introduced in this PR was changed from 1.4 to 9.1.0 (weird versioning?) since only the latter is compatible with Windows 7: https://github.com/aardappel/procrastitracker/commit/7d57768de4ab734b5548a7278ed3d572cd8a0bc4

As far as I can see, that should have no effect on the functionality in this PR.