fraser125 / x360ce

Automatically exported from code.google.com/p/x360ce
0 stars 1 forks source link

Bug in reading Analog +/- button from ini-file. #243

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Assign any button except "Button 1" to "Stick up" in x360ce.App 
2. Using that button has no effect
3. Using "Button 1" results in "0;32767"

The bug is on line 249 and line 253 in Config.cpp. Missing parenthesis makes 
the variable "ret" to contain the result from the comparison instead of the 
result from ReadLongFromFile().

I have attached a patch-file fixing the problem. It is verfied on my system.

Looking at the SVN blame indicates that the bug was introduced in the last 
release (444)

Thanks for a great utility.

What is the expected output? What do you see instead?

What version of x360ce and Windows are you using?
3.3.1.444 on Windows 7 64-bit.

What gamepad are you using? On what version of drivers?
Logitech MOMO Racing USB

Please attach a log file created with Log=1 setting in x360ce.ini file,
and screenshot of x360ce app tab with you gamepad capabilities.

Issue report without Log will be marked as Invalid !

Original issue reported on code.google.com by niklas.g...@gmail.com on 29 Dec 2011 at 10:39

Attachments:

GoogleCodeExporter commented 9 years ago
Have same issue but only 32bit version and problem is with stick up, down, left 
and right. Had to download old version to get this working, too stupid to patch 
it myself.

Original comment by s6dal...@gmail.com on 5 Mar 2012 at 1:18

GoogleCodeExporter commented 9 years ago

Original comment by tapcio on 30 Jan 2013 at 6:31