joystick_drivers icon indicating copy to clipboard operation
joystick_drivers copied to clipboard

Bugfix to support multiple joysticks

Open whoenig opened this issue 3 years ago • 2 comments

The prior initialization/cleanup in the loop prevents that multiple joysticks are working. Since SDL is event-based, it suffices to initialize/cleanup once.

Fixes issue #231.

whoenig avatar Feb 15 '22 10:02 whoenig

I have tested this and the fix works for me, what's the process to get this merged?

KOGDev avatar Mar 21 '22 13:03 KOGDev

Any update on this? It's kind of a shame that we still can't use multiple joysticks in ROS2 without a custom patch like this one...

whoenig avatar Aug 03 '22 16:08 whoenig

I believe that since #247 , this is actually no longer a problem. We now only attempt to SDL_Init once during the constructor.

Given that, I'm going to close this out. But feel free to reopen if this is still a problem for you.

clalancette avatar Sep 14 '23 16:09 clalancette