Event field "which" differs for ControllerDeviceAdded and ControllerDeviceRemoved

[Bastczuak]

Hey,

while implementing the events ControllerDeviceAdded and ControllerDeviceRemoved , I realized that they have different data types for the which field.

One is u32 the other one is i32 https://rust-sdl2.github.io/rust-sdl2/sdl2/event/enum.Event.html#variant.ControllerDeviceAdded.

Is that on purpose or a mistake? Because the official sdl wiki says its always sint32 https://wiki.libsdl.org/SDL_ControllerDeviceEvent.

Greetings

[Cobrand]

It's probably a mistake! Almost all other "which" fields are i32, and the doc says it should be i32, so we can almost safely assume this.

[Bastczuak]

@Cobrand I can do this for you, if you want?!

[Cobrand]

Sure, that would help!

[Bastczuak]

#907 Created a pull request for you.

[mnmaita]

Hey there @Cobrand ! Been using SDL2 for some time and came across the Rust bindings not so long ago. Was taking a look at the repo's Issues and noticed #907 was merged so I guess you could close this issue.