summaryrefslogtreecommitdiffstats
path: root/src/input_common/sdl/sdl_impl.cpp (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #6119 from german77/SDLMappingbunnei2021-04-161-6/+20
|\ | | | | InputCommon: Address mapping and naming issues with SDL2
| * InputCommon: Name properly xbox 360 and one controllers, Fix mappings for Nintendo Pro controllersgerman772021-03-311-6/+20
| |
* | common: Move settings to common from core.bunnei2021-04-151-1/+1
|/ | | | - Removes a dependency on core and input_common from common.
* Revert "Port citra-emu/citra#5123: "SDL: Disable hidapi drivers due to compatibility problems with certain controllers""Morph2021-02-181-7/+0
|
* Merge pull request #3603 from FearlessTobi/port-5123bunnei2021-02-161-0/+7
|\ | | | | Port citra-emu/citra#5123: "SDL: Disable hidapi drivers due to compatibility problems with certain controllers"
| * sdl_joystick: disable the use of the hidapi drivers due to many problems caused by them.Vitor Kiguchi2020-08-301-0/+7
| | | | | | | | | | The main problem is the loss of compatibility with some controllers, but there are also unwanted changes to the behaviour of PS4 controllers (hardcoded lightbar color).
* | Make settings controller image change with controller inputgerman2021-02-061-0/+10
| |
* | sdl_impl: Set the maximum vibration duration to 1 secondMorph2021-01-231-2/+6
| |
* | Merge pull request #5265 from german77/port5509bunnei2020-12-311-2/+45
|\ \ | | | | | | Port citra-emu/citra#5509 "Look at direction of analog axis travel instead of instantaneous sample"
| * | Port citra-emu/citra#5509german2020-12-311-2/+45
| | |
* | | Allow to invert analog axis with right clickgerman2020-12-271-5/+21
|/ /
* | input_common: Add more missing [[maybe_unused]] from #4927.bunnei2020-11-241-1/+2
| |
* | input_common: Treat warnings as errorsLioncash2020-11-221-1/+3
| | | | | | | | | | Migrates over warnings as errors for input common to match how the common library treats warnings as errors.
* | Modify rumble amplificationgerman772020-11-191-2/+1
| |
* | sdl_impl: Pump SDL Events at 1000 HzMorph2020-11-161-1/+1
| |
* | sdl_impl: Revert to the "old" method of mapping sticksMorph2020-11-161-32/+13
| | | | | | | | | | | | Not all controllers have a SDL_GameController binding. This caused controllers not present in the SDL GameController database to have buttons mapped instead of axes. Furthermore, it was not possible to invert the axes when it could be useful such as emulating a horizontal single joycon or other potential cases. This allows us to invert the axes by reversing the order of mapping (vertical, then horizontal).
* | controllers/npad: Remove the old vibration filterMorph2020-11-161-15/+0
| | | | | | | | Previously we used a vibration filter that filters out amplitudes close to each other. It turns out there are cases where this results into vibrations that are too inaccurate. Remove this and move the 100Hz vibration filter (Only allowing a maximum of 100 vibrations per second) from sdl_impl to npad when enable_accurate_vibrations is set to false.
* | input_common: Add VibrationDevice and VibrationDeviceFactoryMorph2020-11-161-16/+58
| | | | | | | | | | | | A vibration device is an input device that returns an unsigned byte as status. It represents whether the vibration device supports vibration or not. If the status returns 1, it supports vibration. Otherwise, it does not support vibration.
* | controllers/npad: Add heuristics to reduce rumble state changesMorph2020-11-161-29/+25
| | | | | | | | | | Sending too many state changes in a short period of time can cause massive performance issues. As a result, we have to use several heuristics to reduce the number of state changes to minimize/eliminate this performance impact while maintaining the quality of these vibrations as much as possible.
* | sdl_impl: Fix controller reconnection issuesMorph2020-10-211-85/+84
| | | | | | | | | | | | | | It turns out that after a controller is disconnected, there is a chance that events from the previous controller are sent/processed after it has been disconnected. This causes the previously disconnected controller to reappear as connected due to GetSDLJoystickBySDLID() emplacing this controller back to the map. Fix this by only returning an SDLJoystick if and only if it exists in the map.
* | sdl_impl: Erase the SDLJoystick entry after removing a controllerMorph2020-10-161-15/+13
| | | | | | | | | | | | | | Previously, disconnecting a controller still leaves a null SDLJoystick entry within the vector of SDLJoysticks mapped by GUID. When a DirectInput device of the same GUID is reconnected, it adds that device to a new port causing non-detectable input. Furthermore, opening the "Configure" menu would cause yuzu to crash since it first tries to resolve the name of a null SDLJoystick entry that was not removed. Resolve this by properly erasing the SDLJoystick entry from the vector.
* | input_common/CMakeLists: Make some warnings errorsLioncash2020-10-161-37/+40
| | | | | | | | | | Makes the input_common code warnings consistent with the rest of the codebase.
* | Merge pull request #4677 from german77/ShakeFromButtonbunnei2020-10-081-0/+190
|\ \ | | | | | | InputCommon: Add random motion input for buttons
| * | Add random motion input to SDLgerman2020-09-261-0/+190
| | |
* | | First implementation of controller rumblegerman2020-09-291-1/+38
|/ /
* | sdl_impl: Reduce allocations in GetButtonMappingForDevice()Lioncash2020-08-291-31/+37
| | | | | | | | These maps can be constexpr arrays of std::pair.
* | sdl_impl: Make use of std::move on std::string where applicableLioncash2020-08-291-3/+3
| | | | | | | | Avoids redundant copies.
* | sdl_impl: Make use of insert_or_assign() where applicableLioncash2020-08-291-14/+18
| | | | | | | | Avoids churning ParamPackage instances.
* | sdl_impl: Prevent type truncation in BuildAnalogParamPackageForButton() default argumentsLioncash2020-08-291-1/+1
| | | | | | | | | | We need to add the 'f' suffix to make the right hand side a float and not a double.
* | sdl_impl: Simplify make_tuple callLioncash2020-08-291-1/+1
| | | | | | | | | | | | | | The purpose of make_tuple is that you don't need to explicitly type out the types of the things that comprise said tuple. Given this just returns default values, we can simplify this a bit.
* | sdl_impl: Mark FromEvent() as a const member functionLioncash2020-08-291-2/+2
|/ | | | This doesn't modify internal member state, so it can be marked as const.
* input_common/main: Remove unnecessary headersLioncash2020-08-291-1/+1
|
* input_common: Fix directional deadzone valuesMorph2020-08-261-1/+1
| | | | The hardware tested value is 0.5 which translates to SHRT_MAX / 2
* Address feedbackMorph2020-08-261-32/+20
|
* Project Mjölnir: Part 1Morph2020-08-261-90/+313
| | | | | Co-authored-by: James Rowe <jroweboy@gmail.com> Co-authored-by: Its-Rei <kupfel@gmail.com>
* Remove UI changesameerj2020-08-101-1/+1
| | | | This PR is now only the Analog devices handling the range value defaulting at 100%
* undo unnecessary newlines, slider range 50-150Ameer2020-08-101-1/+1
|
* Add range slider for analog sticksAmeer2020-08-101-11/+14
|
* CMakeLists: Specify -Wextra on linux buildsLioncash2020-04-161-0/+1
| | | | | | | | | | | Allows reporting more cases where logic errors may exist, such as implicit fallthrough cases, etc. We currently ignore unused parameters, since we currently have many cases where this is intentional (virtual interfaces). While we're at it, we can also tidy up any existing code that causes warnings. This also uncovered a few bugs as well.
* Moved analog direction logic to sdl_implCJBok2020-01-151-0/+16
|
* input_common/sdl/sdl_impl: Correct logging string in SDLState constructorLioncash2019-06-031-1/+1
| | | | | | If this path was ever taken, a runtime exception would occur due to the lack of a formatting specifier to insert the error code into the format string.
* input_common/sdl/sdl_impl: Move documentation comments to header where applicableLioncash2019-06-031-7/+0
| | | | | Places the documentation comments with the rest of SDLState's member function documentation.
* input_common/sdl/sdl_impl: Amend names for axes for SDLAnalogPollerLioncash2019-06-031-13/+13
| | | | Adds another underscore to clearly indicate the axis names.
* input_common/sdl/sdl_impl: Mark variables const where applicableLioncash2019-06-031-10/+11
| | | | | Make it explicit that these aren't modified elsewhere (either through functions by reference, or by other operations).
* input_common/sdl/sdl_impl: Mark SDLEventToButtonParamPackage() as staticLioncash2019-06-031-1/+1
| | | | | | Its prototype declared at the top of the translation unit contains the static qualifier, so the function itself should also contain it to make it a proper internally linked function.
* input_common/sdl/sdl_impl: Convert reinterpret_cast into a static_castLioncash2019-06-031-2/+4
| | | | It's valid to static_cast a void pointer back into its proper type.
* input_common/sdl/sdl_impl: Use insert_or_assign() where applicableLioncash2019-06-031-3/+3
| | | | | Same behavior, but without a potential need to unnecessarily default construct a value.
* input_common/sdl/sdl_impl: Simplify SDL_Joystick deleter handlingLioncash2019-06-031-15/+14
| | | | | | | | | The deleter can just be set in the constructor and maintained throughout the lifetime of the object. If a contained pointer is null, then the deleter won't execute, so this is safe to do. We don't need to swap it out with a version of a deleter that does nothing.
* input_common/sdl/sdl_impl: Resolve two sign conversion warningsLioncash2019-06-031-10/+16
| | | | Silences the final two warnings in SDL code.
* input_common/sdl: Remove unused header includes and forward declarationsLioncash2019-06-031-2/+0
| | | | | Gets rid of a few unnecessary inclusion dependencies. It also uncovered a few indirect inclusion dependencies being relied upon.
* input_common/sdl/sdl_impl: Use nested namespace specifiers where applicableLioncash2019-06-031-5/+2
|
* input_common/sdl/sdl_impl: Silence sign conversion warningsLioncash2019-05-311-3/+3
| | | | Makes the conversions explicit, as opposed to implicit.
* general: Use deducation guides for std::lock_guard and std::unique_lockLioncash2019-04-011-12/+14
| | | | | | | Since C++17, the introduction of deduction guides for locking facilities means that we no longer need to hardcode the mutex type into the locks themselves, making it easier to switch mutex types, should it ever be necessary in the future.
* input_common/sdl: Correct return values within implementations of GetPollers()Lioncash2019-03-181-1/+3
| | | | | In both cases, we weren't actually returning anything, which is undefined behavior.
* input_common/sdl: Use a type alias to shorten declaration of GetPollersLioncash2019-03-181-3/+2
| | | | Just makes the definitions a little bit more tidy.
* input_common/sdl_impl: Make lambda capture more specific in SDLState constructorLioncash2019-03-171-1/+1
| | | | | We don't need to universally capture by reference. We specifically just need to capture the this pointer.
* input_common/sdl_impl: Remove unnecessary std::chrono::duration constructionLioncash2019-03-171-1/+1
| | | | Specifying the time unit itself is sufficient here.
* input_common/sdl_impl: Remove unused variable in SDLState constructorLioncash2019-03-171-1/+0
|
* fixup! Joystick: Allow for background events; Add deadzone to SDLAnalogB3n302019-03-021-6/+17
|
* input/sdl: lock map mutex after SDL callWeiyi Wang2019-03-021-11/+17
| | | | Any SDL invocation can call the even callback on the same thread, which can call GetSDLJoystickBySDLID and eventually cause double lock on joystick_map_mutex. To avoid this, lock guard should be placed as closer as possible to the object accessing code, so that any SDL invocation is with the mutex unlocked
* Input: Remove global variables from SDL InputJames Rowe2019-03-021-85/+108
| | | | | | | | | Changes the interface as well to remove any unique methods that frontends needed to call such as StartJoystickEventHandler by conditionally starting the polling thread only if the frontend hasn't started it already. Additionally, moves all global state into a single SDLState class in order to guarantee that the destructors are called in the proper order
* Input: Copy current SDL.h/cpp files to implJames Rowe2019-03-021-0/+629
This should make reviewing much easier as you can then see what changed happened between the old file and the new one