Sam Lantinga

Results 1875 comments of Sam Lantinga

The new entry has an identical GUID to the existing one, but uses a completely different mapping: old: ``` "03000000550900001472000011010000,NVIDIA Controller v01.04,a:b0,b:b1,back:b14,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,guide:b17,leftshoulder:b4,leftstick:b7,lefttrigger:a3,leftx:a0,lefty:a1,rightshoulder:b5,rightstick:b8,righttrigger:a4,rightx:a2,righty:a5,start:b6,x:b2,y:b3,", ``` new (without CRC): ``` "03000000550900001472000011010000,NVIDIA Controller v01.04,a:b0,b:b1,back:b7,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,guide:b4,leftshoulder:b5,leftstick:b10,lefttrigger:a3,leftx:a0,lefty:a1,misc1:b8,rightshoulder:b6,rightstick:b11,righttrigger:a4,rightx:a2,righty:a5,start:b9,x:b2,y:b3,",...

I confirmed that the existing mapping was mostly correct for my NVIDIA Shield Controller v1.04, and added an entry for Bluetooth mode. This mapping works for both v1.25 and v1.35...

If we have the same model, maybe it's our Linux kernel that's different? What is the output of `cat /etc/os-release; uname -a`?

We probably need to divide the values coming in by 120. Take a look at the WHEEL_DELTA code in SDL_windowsevents.c for reference.

@thatcosmonaut, is implementing MSAA simply a matter of setting `SampleDesc.Count` to a specific value, or is there more to it than that?

Looking at the haptic code it looks like it would benefit from the same speedup approach that we used with joysticks. I'll throw this on the 3.2 list. Feel free...

We are scoping work for the SDL 3.2.0 release, so please let us know if this is a showstopper for you.

> I noticed a similar bug where SDL would report two events and two unique SDL_JoystickID's for plugging in a single controller. > > Repro steps: > > * Start...

> SDL_GUID 03004d6e5e040000ea02000000007701 for SDL_JoystickID 4 Joypad 1 connected. SDL_GUID 0300938d5e040000ff02000000007200 for SDL_JoystickID 5 Joypad 2 connected. Can you debug and see why SDL_IsXInputDevice() is not returning true in SDL_windows_gaming_input.c?

It turns out this was pretty easy to do, can you try https://github.com/libsdl-org/SDL/pull/14431 and see if that works for you?