We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] Same patch as previous, but for SDL version 1.

SDL1_remove_dead_zone_correctly_patch.txt (text/plain), 3.70 KB, created by pj5085 on 2020-08-27 04:02:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: pj5085

Created: 2020-08-27 04:02:29 UTC

Size: 3.70 KB

patch

obsolete

>--- SDL-1.2.15/src/joystick/linux/SDL_sysjoystick.c	2012-01-19 01:30:06.000000000 -0500
>+++ SDL-1.2.15-modified/src/joystick/linux/SDL_sysjoystick.c	2020-08-26 22:13:15.212757829 -0400
>@@ -25,6 +25,7 @@
> 
> /* This is the system specific header for the SDL joystick API */
> 
>+#include <math.h>
> #include <sys/stat.h>
> #include <unistd.h>
> #include <fcntl.h>
>@@ -290,7 +291,8 @@
> 	Uint8 abs_map[ABS_MAX];
> 	struct axis_correct {
> 		int used;
>-		int coef[3];
>+		int minimum;
>+		int maximum;
> 	} abs_correct[ABS_MAX];
> #endif
> };
>@@ -670,7 +672,7 @@
> 
> static SDL_bool EV_ConfigJoystick(SDL_Joystick *joystick, int fd)
> {
>-	int i, t;
>+	int i;
> 	unsigned long keybit[NBITS(KEY_MAX)] = { 0 };
> 	unsigned long absbit[NBITS(ABS_MAX)] = { 0 };
> 	unsigned long relbit[NBITS(REL_MAX)] = { 0 };
>@@ -724,16 +726,8 @@
> 				    joystick->hwdata->abs_correct[i].used = 0;
> 				} else {
> 				    joystick->hwdata->abs_correct[i].used = 1;
>-				    joystick->hwdata->abs_correct[i].coef[0] =
>-					(absinfo.maximum + absinfo.minimum) / 2 - absinfo.flat;
>-				    joystick->hwdata->abs_correct[i].coef[1] =
>-					(absinfo.maximum + absinfo.minimum) / 2 + absinfo.flat;
>-				    t = ((absinfo.maximum - absinfo.minimum) / 2 - 2 * absinfo.flat);
>-				    if ( t != 0 ) {
>-					joystick->hwdata->abs_correct[i].coef[2] = (1 << 29) / t;
>-				    } else {
>-					joystick->hwdata->abs_correct[i].coef[2] = 0;
>-				    }
>+				    joystick->hwdata->abs_correct[i].minimum = absinfo.minimum;
>+				    joystick->hwdata->abs_correct[i].maximum = absinfo.maximum;
> 				}
> 				++joystick->naxes;
> 			}
>@@ -1055,20 +1049,46 @@
> 
> 	correct = &joystick->hwdata->abs_correct[which];
> 	if ( correct->used ) {
>-		if ( value > correct->coef[0] ) {
>-			if ( value < correct->coef[1] ) {
>-				return 0;
>-			}
>-			value -= correct->coef[1];
>-		} else {
>-			value -= correct->coef[0];
>-		}
>-		value *= correct->coef[2];
>-		value >>= 14;
>+
>+          int original_value = value;
>+          //if (value < -40) original_value -= 40;
>+          //if (value > +40) original_value += 40;
>+
>+          float ideal_minimum = -32767;
>+          float ideal_maximum = +32767;
>+          float ideal_range = ideal_maximum - ideal_minimum;
>+          float actual_range = (correct->maximum - correct->minimum);
>+          float coefficient = ideal_range / actual_range;
>+          float new_float = (original_value - correct->minimum) * coefficient + ideal_minimum;
>+          int new_integer = round(new_float);
>+
>+          // At least one gamepad has a minimum of 0 and a maximum is 255,
>+          // which makes it's center point a non-integer, 127.5.  Thus it
>+          // must report either 127 or 128, neither of which is center.
>+
>+          // At least one analog controller with the same range returns 128
>+          // easily when at center, but seems completely incapable of
>+          // returning 127 no matter how carefully the stick is moved.
>+
>+          // So here we detect if the axis' range does not have a center
>+          // point, and if so, and if the reported position is on either
>+          // side of center, we report that it is actually at the center.
>+
>+          if ((correct->maximum - correct->minimum) & 1) {
>+            int lower_center = (correct->maximum - correct->minimum - 1) / 2;
>+            int higher_center = (correct->maximum - correct->minimum + 1) / 2;
>+            if (original_value == lower_center) new_integer = 0;
>+            if (original_value == higher_center) new_integer = 0;
>+          };
>+
>+          if (which == 2) printf("original=%d, new=%d\n", original_value, new_integer);
>+
>+          value = new_integer;
>+
> 	}
> 
> 	/* Clamp and return */
>-	if ( value < -32768 ) return -32768;
>+	if ( value < -32767 ) return -32767;
> 	if ( value >  32767 ) return  32767;
> 
> 	return value;

Actions: View | Diff

Attachments on bug 5241: 4423 | 4446 | 4450 | 4451 | 4452 | 4619 | 4620

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.