summaryrefslogtreecommitdiffstats
path: root/src/input_common (follow)
Commit message (Collapse)AuthorAgeFilesLines
* input_common: Eliminate most global stateLioncash2020-08-272-153/+227
| | | | | | | | | | | | | Abstracts most of the input mechanisms under an InputSubsystem class that is managed by the frontends, eliminating any static constructors and destructors. This gets rid of global accessor functions and also allows the frontends to have a more fine-grained control over the lifecycle of the input subsystem. This also makes it explicit which interfaces rely on the input subsystem instead of making it opaque in the interface functions. All that remains to migrate over is the factories, which can be done in a separate change.
* input_common/main: Add "/Mouse" to the display nameMorph2020-08-271-1/+1
|
* input_common: Fix directional deadzone valuesMorph2020-08-262-2/+2
| | | | The hardware tested value is 0.5 which translates to SHRT_MAX / 2
* Address feedbackMorph2020-08-265-44/+32
|
* Project Mjölnir: Part 1Morph2020-08-2611-102/+798
| | | | | Co-authored-by: James Rowe <jroweboy@gmail.com> Co-authored-by: Its-Rei <kupfel@gmail.com>
* General: Tidy up clang-format warnings part 2Lioncash2020-08-131-4/+2
|
* Remove UI changesameerj2020-08-102-2/+2
| | | | This PR is now only the Analog devices handling the range value defaulting at 100%
* Add range slider functionality for gc adapterameerj2020-08-101-7/+7
|
* undo unnecessary newlines, slider range 50-150Ameer2020-08-101-1/+1
|
* Add range slider for analog sticksAmeer2020-08-101-11/+14
|
* GCAdapter: only join worker thread if running & joinableBrian J. Tarricone2020-07-261-1/+3
|
* Merge pull request #4418 from lioncash/udp-warnbunnei2020-07-261-1/+0
|\ | | | | udp/client: Remove unused boost include
| * udp/client: Remove unused boost includeLioncash2020-07-251-1/+0
| | | | | | | | Also silences a deprecation warning from boost on Clang/GCC.
* | gc_adapter: Resolve C++20 deprecation warningLioncash2020-07-251-1/+1
| |
* | gc_poller: Resolve -Wsign-compare warningLioncash2020-07-251-1/+2
| |
* | gc_poller: Resolve -Wredundant-move warningLioncash2020-07-251-2/+1
|/
* Fix axis thresholding while pollingameerj2020-07-191-5/+2
| | | | axes were very sensitive when mapping controls.
* std::size_t where appropriate, make error message more clear if can't readameerj2020-07-171-3/+4
|
* Refactor adapter codeAmeer2020-07-162-179/+44
|
* Rebase to masterAmeer2020-07-147-31/+48
|\
| * Fix crash if gc configured but adapter not connectedAmeer2020-07-142-15/+25
| |
| * Merge pull request #4314 from lioncash/input-warnbunnei2020-07-144-11/+15
| |\ | | | | | | gcadapter: Tidy up compiler warnings
| | * gc_poller: Mark GCButtonFactory::GetNextInput() as constLioncash2020-07-122-2/+2
| | | | | | | | | | | | This doesn't modify class instance state.
| | * gc_poller: Get rid of undefined behavior in Create()Lioncash2020-07-121-0/+4
| | | | | | | | | | | | Ensures that the function always has returns in all control paths.
| | * gc_poller: Silence sign conversion warningsLioncash2020-07-121-1/+1
| | |
| | * gc_adapter: Remove deprecated usage of = in lambda capturesLioncash2020-07-121-1/+1
| | | | | | | | | | | | | | | | | | | | | It's deprecated in C++20 to use = to capture the this pointer. Instead, we can simply pass this as an argument to the thread constructor.
| | * gc_adapter: Silence sign conversion warningsLioncash2020-07-122-7/+7
| | |
| * | Merge pull request #4315 from lioncash/udp-warnbunnei2020-07-141-1/+1
| |\ \ | | | | | | | | udp: Silence a C++20 deprecation warning
| | * | udp: Silence a C++20 deprecation warningLioncash2020-07-121-1/+1
| | |/ | | | | | | | | | | | | C++20 deprecates using the = lambda capture to implicitly capture the this pointer. Instead, we must specify it explicitly.
| * | input_common: drop unused libusb.h includeAmeer2020-07-141-1/+0
| | | | | | | | | | | | Remnant of an early implementation.
| * | input_common: make libusb private to gc_adapterJan Beich2020-07-133-3/+7
| | |
| * | cmake: pass libusb include directory as wellJan Beich2020-07-091-0/+1
| |/ | | | | | | | | | | | | In file included from src/input_common/gcadapter/gc_adapter.cpp:8: src/./input_common/gcadapter/gc_adapter.h:11:10: fatal error: 'libusb.h' file not found #include <libusb.h> ^~~~~~~~~~
* | Break out of scan loop if can't find adapter on first runAmeer2020-07-101-0/+3
| |
* | Rebase to master, fix merge conflictsAmeer2020-07-092-10/+30
|\|
| * Merge pull request #4266 from jbeich/freebsdbunnei2020-07-081-0/+1
| |\ | | | | | | gcadapter: unbreak build on FreeBSD
| | * input_common/gcadapter: add missing C++11 header required by libc++Jan Beich2020-07-071-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In file included from src/input_common/gcadapter/gc_adapter.cpp:8: src/./input_common/gcadapter/gc_adapter.h:77:10: error: no template named 'unordered_map' in namespace 'std' std::unordered_map<int, bool> buttons; ~~~~~^ src/./input_common/gcadapter/gc_adapter.h:78:10: error: no template named 'unordered_map' in namespace 'std' std::unordered_map<int, u16> axes; ~~~~~^
| * | Address comments for better clarity/signed dev countAmeer2020-07-081-11/+15
| | |
| * | Add more libusb error checksAmeer2020-07-081-3/+18
| |/
* | Address PR feedback, fix axis button thresholdingAmeer2020-07-073-58/+22
| |
* | Brace the code! Fix compile error due to class member construction orderAmeer2020-07-072-15/+31
| |
* | Recalibrate reconnected controllersAmeer2020-07-071-0/+5
| |
* | Save origin state of GC controller analog features, compare against origin for input detectionAmeer2020-07-073-28/+72
|/
* Fix for always firing triggers on some controllers, trigger threshold more universalAmeer2020-07-042-5/+5
|
* Address lioncash feedback: Log formatting, extern const PadButtonArray, little touch upsAmeer2020-07-033-24/+32
|
* Fix unnecessary diffsAmeer2020-07-023-3/+3
|
* Add LR triggers as axes, half press to initiate a press, add GC axis id in config, clarify some code blocks for better readabilityAmeer2020-07-022-10/+34
|
* Reset adapter state on init, fixes errors relating driver hang from unexpected unplugAmeer2020-07-012-0/+9
|
* Address feedback regarding increments, const vars, and general cleanupAmeer2020-06-302-24/+21
|
* fix implicit conversion of size_t type to intAmeer2020-06-301-1/+1
|
* left const auto&, comment punctuation.Ameer J2020-06-271-2/+2
| | | Co-authored-by: Morph <39850852+Morph1984@users.noreply.github.com>
* const& to button in button arrayAmeer J2020-06-261-2/+2
| | | Co-authored-by: VolcaEM <63682805+VolcaEM@users.noreply.github.com>
* Stop reading loop if error is encounteredAmeer2020-06-261-4/+4
|
* padbutton enum class and struct initiailizationAmeer2020-06-243-36/+32
|
* cleanup check access, read, and factory GetNextInput funcs. Use size rather than magic numberAmeer2020-06-234-151/+101
|
* Fix deallocation of GC AdapterAmeer2020-06-233-4/+10
|
* std::array and const reference passing of non-trivial objectsAmeer2020-06-232-14/+13
|
* Update src/input_common/main.cppameerj2020-06-221-1/+1
| | | | | | Co-authored-by: LC <mathew1800@gmail.com> update libusb submodule (hopefully windows build error fixed)
* Tidy up the pointers, use pair over tuple where appropriateAmeer2020-06-224-24/+22
|
* fix for sleep using stlAmeer2020-06-221-1/+3
|
* shared_ptr for the GC adapter class, constexpr constantsAmeer2020-06-225-66/+52
|
* std::arrays where appropriate, clear q in adapter class, other touch upsAmeer2020-06-225-27/+15
|
* fix include threadAmeer2020-06-221-0/+1
|
* Singleton GC Adapter class, remove globals, fix naming conventionAmeer2020-06-227-202/+259
| | | | | | | | Fix clang formatting Manual fix for configure_input_player formatting Add missing lib usb cmake command
* Clang FormattingAmeer2020-06-216-77/+99
|
* Cleanup after linterAmeer2020-06-219-167/+98
|
* GC Adapter ImplementationAmeer2020-06-2115-100/+1039
|
* input_common/motion_emu: Remove redundant moveMerryMage2020-06-191-1/+1
| | | | Named return value optimization automatically applies here.
* input_common/keyboard: Remove redundant moveMerryMage2020-06-191-1/+1
| | | | Named return value optimization automatically applies here.
* input_common: fix build when SDL2 is disabledFearlessTobi2020-05-021-0/+6
|
* Merge pull request #3673 from lioncash/extrabunnei2020-04-171-0/+1
|\ | | | | CMakeLists: Specify -Wextra on linux builds
| * 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.
* | input_common: Use the CMake target instead of the variable.Markus Wick2020-04-161-1/+1
|/
* Merge pull request #3525 from FearlessTobi/linux-compile-errorbunnei2020-03-211-1/+1
|\ | | | | input_common/udp: Fix Linux build by using a backwards compatible way of error checking
| * input_common/udp: Fix Linux build by using a backwards compatible way of error checkingFearlessTobi2020-03-171-1/+1
| | | | | | | | | | | | | | Should fix https://github.com/yuzu-emu/yuzu/issues/3487. error_code::failed is a function which has been introduced in Boost 1.69. This version of boost hasn't landed in most major distros yet.
* | input_common/udp: Fix clang build issuesReinUsesLisp2020-03-181-1/+2
|/
* input/udp - Add minor error handling to prevent bad input from crashingJames Rowe2020-03-041-4/+13
|
* input/udp - Dont log on invalid packet receivedJames Rowe2020-03-041-1/+0
|
* analog_from_button get direction implementationCJBok2020-02-181-0/+14
|
* input_common/udp: Ensure that UDP is shut down within Shutdown()Lioncash2020-02-031-0/+1
| | | | Previously the UDP backend would never actually get shut down.
* input_common/udp: Add missing override specifiersLioncash2020-02-031-2/+2
| | | | | Prevents trivial warnings and ensures interfaces are properly maintained between the base class.
* input_common/udp: std::move SocketCallback instances where applicableLioncash2020-02-031-2/+2
| | | | | | std::function is allowed to heap allocate if the size of the captures associated with each lambda exceed a certain threshold. This prevents potentially unnecessary reallocations from occurring.
* input_common/udp: std::move shared_ptr within Client constructorLioncash2020-02-031-1/+1
| | | | | Gets rid of a trivially avoidable atomic reference count increment and decrement.
* udp/client: Replace deprecated from_string() call with make_address_v4()Lioncash2020-02-031-2/+2
| | | | Future-proofs code if boost is ever updated.
* input_common/udp: Silence -Wreorder warning for SocketLioncash2020-02-031-4/+3
| | | | | Amends the constructor initializer list to specify the order of its elements in the same order that initialization would occur.
* input_common/udp: Remove unnecessary inclusionsLioncash2020-02-034-8/+4
|
* input_common/udp: Add missing header guardLioncash2020-02-031-0/+2
|
* Merge pull request #3284 from CJBok/hid-fixbunnei2020-02-011-0/+16
|\ | | | | hid: Fix analog sticks directional states
| * Moved analog direction logic to sdl_implCJBok2020-01-151-0/+16
| |
* | Address second part of review commentsFearlessTobi2020-01-233-13/+17
| |
* | Address review commentsFearlessTobi2020-01-234-65/+72
| |
* | Input: UDP Client to provide motion and touch controlsfearlessTobi2020-01-238-4/+846
|/ | | | | | | | An implementation of the cemuhook motion/touch protocol, this adds the ability for users to connect several different devices to citra to send direct motion and touch data to citra. Co-Authored-By: jroweboy <jroweboy@gmail.com>
* 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-032-7/+6
| | | | | 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-033-11/+5
| | | | | 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-013-21/+23
| | | | | | | 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-182-2/+6
| | | | | 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-183-11/+9
| | | | 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-027-800/+201
| | | | | | | | | 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-022-0/+680
| | | | | This should make reviewing much easier as you can then see what changed happened between the old file and the new one
* common/math_util: Move contents into the Common namespaceLioncash2019-02-271-2/+2
| | | | | These types are within the common library, so they should be within the Common namespace.
* common/vector_math: Move Vec[x] types into the Common namespaceLioncash2019-02-271-11/+11
| | | | | These types are within the common library, so they should be using the Common namespace.
* common/quaternion: Move Quaternion into the Common namespaceLioncash2019-02-271-4/+4
| | | | | Quaternion is within the common library, so it should be using the Common namespace.
* Port #4141 from citra: Joystick hotplug support (#1275)Tobias2018-09-114-88/+322
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Joystick hotplug support (#4141) * use SDL_PollEvent instead of SDL_JoystickUpdate Register hot plugged controller by GUID if they were configured in a previous session * Move SDL_PollEvent into its own thread * Don't store SDLJoystick pointer in Input Device; Get pointer on each GetStatus call * Fix that joystick_list gets cleared after SDL_Quit * Add VirtualJoystick for InputDevices thats never nullptr * fixup! Add VirtualJoystick for InputDevices thats never nullptr * fixup! fixup! Add VirtualJoystick for InputDevices thats never nullptr * Remove SDL_GameController, make SDL_Joystick* unique_ptr * fixup! Remove SDL_GameController, make SDL_Joystick* unique_ptr * Adressed feedback; fixed handling of same guid reconnects * fixup! Adressed feedback; fixed handling of same guid reconnects * merge the two joystick_lists into one * make SDLJoystick a member of VirtualJoystick * fixup! make SDLJoystick a member of VirtualJoystick * fixup! make SDLJoystick a member of VirtualJoystick * fixup! fixup! make SDLJoystick a member of VirtualJoystick * SDLJoystick: Addressed review comments * Address one missed review comment
* input_common: Use std::move where applicableLioncash2018-08-032-5/+6
| | | | Avoids unnecessary atomic reference count increments and decrements
* input_common: Add missing override specifiersLioncash2018-08-033-14/+2
|
* Rename logging macro back to LOG_*James Rowe2018-07-031-3/+3
|
* input_common: Move old logging macros over to fmt-capable onesLioncash2018-04-271-3/+3
|
* math_util: Remove the Clamp() functionLioncash2018-04-201-2/+3
| | | | | C++17 adds clamp() to the standard library, so we can remove ours in favor of it.
* Format: Run the new clang format on everythingJames Rowe2018-01-211-1/+2
|
* Merge pull request #84 from lioncash/cmakebunnei2018-01-181-19/+12
|\ | | | | CMakeLists: Derive the source directory grouping from targets themselves
| * CMakeLists: Derive the source directory grouping from targets themselvesLioncash2018-01-181-19/+12
| | | | | | | | | | Removes the need to store to separate SRC and HEADER variables, and then construct the target in most cases.
* | input_common/sdl: Silence a -Wpessimizing-move warningLioncash2018-01-181-1/+1
|/
* Adding meumart's Citra SDL Joystick support. Citra PR #3116muemart2018-01-164-1/+248
|
* motion_emu: fix initialization orderwwylele2017-08-221-1/+4
|
* Merge pull request #2861 from wwylele/motion-refactorJames Rowe2017-08-205-2/+231
|\ | | | | Refactor MotionEmu into a InputDevice
| * motion_emu: no need to include thread in headerwwylele2017-08-192-2/+7
| |
| * move MotionEmu from core/frontend to input_common as a InputDevicewwylele2017-08-115-2/+226
| |
* | Fix Spelling/English mistakesDave Leaver2017-08-131-1/+1
|/
* Fix some spelling mistakesdanzel2017-08-111-1/+1
|
* CMake: Define an interface target for SDL2 definitionsYuri Kunde Schlesner2017-05-281-2/+1
|
* CMake: Correct inter-module dependencies and library visibilityYuri Kunde Schlesner2017-05-281-3/+3
| | | | | | | | | | Modules didn't correctly define their dependencies before, which relied on the frontends implicitly including every module for linking to succeed. Also changed every target_link_libraries call to specify visibility of dependencies to avoid leaking definitions to dependents when not necessary.
* input_common/sdl: add support for binding button to axiswwylele2017-04-171-4/+53
|
* citra-qt: release all buttons when render window focus is lostwwylele2017-03-172-0/+13
| | | | credit to @Hawkheart for the original idea
* InputCommon: add SDL joystick supportwwylele2017-03-014-0/+241
|
* InputCommon: add AnalogFromButtonwwylele2017-03-015-0/+113
|
* InputCommon: add Keyboardwwylele2017-03-015-0/+202