Don't send controller gyro and accelerometer updates if the values haven't changed.

This keeps SDL_PollEvent() from returning data continuously when polling controller state.
This commit is contained in:
Sam Lantinga 2020-11-17 22:09:52 -08:00
parent 463d84a860
commit 51ea6a6d44
1 changed files with 16 additions and 16 deletions

View File

@ -2569,26 +2569,26 @@ int SDL_PrivateJoystickSensor(SDL_Joystick *joystick, SDL_SensorType type, const
if (sensor->type == type) { if (sensor->type == type) {
if (sensor->enabled) { if (sensor->enabled) {
/* Allow duplicate events, for things like gyro updates */
/* Update internal sensor state */
num_values = SDL_min(num_values, SDL_arraysize(sensor->data)); num_values = SDL_min(num_values, SDL_arraysize(sensor->data));
SDL_memcpy(sensor->data, data, num_values*sizeof(*data)); if (SDL_memcmp(data, sensor->data, num_values*sizeof(*data)) != 0) {
/* Post the event, if desired */ /* Update internal sensor state */
posted = 0; SDL_memcpy(sensor->data, data, num_values*sizeof(*data));
/* Post the event, if desired */
#if !SDL_EVENTS_DISABLED #if !SDL_EVENTS_DISABLED
if (SDL_GetEventState(SDL_CONTROLLERSENSORUPDATE) == SDL_ENABLE) { if (SDL_GetEventState(SDL_CONTROLLERSENSORUPDATE) == SDL_ENABLE) {
SDL_Event event; SDL_Event event;
event.type = SDL_CONTROLLERSENSORUPDATE; event.type = SDL_CONTROLLERSENSORUPDATE;
event.csensor.which = joystick->instance_id; event.csensor.which = joystick->instance_id;
event.csensor.sensor = type; event.csensor.sensor = type;
num_values = SDL_min(num_values, SDL_arraysize(event.csensor.data)); num_values = SDL_min(num_values, SDL_arraysize(event.csensor.data));
SDL_memset(event.csensor.data, 0, sizeof(event.csensor.data)); SDL_memset(event.csensor.data, 0, sizeof(event.csensor.data));
SDL_memcpy(event.csensor.data, data, num_values*sizeof(*data)); SDL_memcpy(event.csensor.data, data, num_values*sizeof(*data));
posted = SDL_PushEvent(&event) == 1; posted = SDL_PushEvent(&event) == 1;
} }
#endif /* !SDL_EVENTS_DISABLED */ #endif /* !SDL_EVENTS_DISABLED */
}
} }
break; break;
} }