summaryrefslogtreecommitdiff
path: root/src/input_common (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7735 from german77/udp_batteryGravatar bunnei2022-01-222-0/+25
|\ | | | | input_common: Report battery for UDP controllers
| * input_common: Report battery for UDP controllersGravatar Narr the Reg2022-01-172-0/+25
| |
* | Merge pull request #7715 from gidoly/patch-4Gravatar bunnei2022-01-181-2/+2
|\ \ | | | | | | Xbox controller default name nit pick
| * | Xbox controller default name nit pickGravatar gidoly2022-01-161-2/+2
| |/ | | | | Discord User moon lacer pointed us that official name is 'Xbox' not 'XBox'
* | Merge pull request #7725 from german77/mouse_in_motionGravatar bunnei2022-01-183-1/+47
|\ \ | | | | | | input_common: Reintroduce motion from mouse and use button names
| * | input_common: Reintroduce motion from mouse and use button namesGravatar german772022-01-163-1/+47
| |/
* | Merge pull request #7730 from v1993/patch-6Gravatar Mai M2022-01-171-1/+1
|\ \ | | | | | | input_common: nitpick about SetHatButton usage
| * | input_common: nitpick about SetHatButton usageGravatar Valeri2022-01-171-1/+1
| |/
* / input_common: fix copy-paste errorGravatar Valeri2022-01-171-1/+1
|/ | | Found by static analysis with PVS-Studio.
* Change default name for ps controllersGravatar gidoly2022-01-151-0/+6
| | | | Minor nitpick Code is from narr
* Merge pull request #7633 from german77/hotkeysGravatar bunnei2022-01-113-4/+7
|\ | | | | yuzu: Add controller hotkeys
| * yuzu: Add controller hotkeysGravatar german772022-01-063-4/+7
| |
* | Merge pull request #7687 from german77/tas_handleGravatar Morph2022-01-091-7/+24
|\ \ | | | | | | input_common: Handle errors on TAS scripts
| * | input_common: Handle errors on TAS scriptsGravatar german772022-01-081-7/+24
| |/
* | Merge pull request #7682 from german77/udp_fixGravatar bunnei2022-01-081-8/+16
|\ \ | | | | | | input_common: Fix UDP controller mappings
| * | input_common: Fix udp motion not automapping to both sidesGravatar german772022-01-071-8/+16
| |/
* / input_common: Use accelerometer data for mappingGravatar german772022-01-061-2/+10
|/
* Merge pull request #7302 from VPeruS/check-deadlockGravatar bunnei2021-12-172-43/+52
|\ | | | | [input_common] Fixed thread hang
| * [input_common] Move variable declaration closer to usageGravatar vperus2021-12-171-2/+2
| | | | | | | | | | | | | | | | MSVC supplied with VS2022 generates "warning C4189: 'CALIBRATION_THRESHOLD': local variable is initialized but not referenced" which is treated as an error. Circumvent it by moving constexpr variable directly into body of lambda function.
| * Revert of b01aa72Gravatar vperus2021-11-291-35/+39
| | | | | | | | Caused worker_thread to be stuck in Stage1Completed state until job's destruction.
| * [input_common] Add completion test for CalibrationConfigurationJobGravatar vperus2021-11-291-8/+13
| |
* | Merge pull request #7581 from lioncash/input-ifaceGravatar Narr the Reg2021-12-143-55/+70
|\ \ | | | | | | common/input: Avoid numerous large copies of CallbackStatus
| * | common/input: Avoid numerous large copies of CallbackStatusGravatar Lioncash2021-12-132-35/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | CallbackStatus instances aren't the cheapest things to copy around (relative to everything else), given that they're currently 520 bytes in size and are currently copied numerous times when callbacks are invoked. Instead, we can pass the status by const reference to avoid all the copying.
| * | input_poller: Add missing override specifiersGravatar Lioncash2021-12-131-20/+19
| | |
* | | Merge pull request #7577 from v1993/patch-2Gravatar Narr the Reg2021-12-141-3/+4
|\ \ \ | | | | | | | | input/SDL: Update SDL hints
| * | | input/SDL: Update SDL hintsGravatar Valeri2021-12-141-3/+4
| | | | | | | | | | | | | | | | | | | | SDL_HINT_JOYSTICK_HIDAPI_SWITCH_HOME_LED is no longer needed thanks to new default in SDL 2.0.18. SDL_HINT_JOYSTICK_HIDAPI_XBOX is reported to cause conflicts with native driver Xbox driver on Linux, and Xbox controllers don't benefit from hidapi anyways.
* | | | input_mapping: Amend specification of parametersGravatar Lioncash2021-12-131-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | param tags are supposed to specify the parameter name without any quoting. Silences several -Wdocumentation warnings.
* | | | input_poller: Remove several unnecessary @param tagsGravatar Lioncash2021-12-131-106/+106
| |/ / |/| | | | | | | | | | | | | | Silences quite a bit of -Wdocumentation warnings, given the @param tag is only intended to be used to identify function parameters, not what it contains.
* | | Merge pull request #7575 from lioncash/inputGravatar bunnei2021-12-1317-110/+107
|\ \ \ | | | | | | | | input_engine: Minor object churn cleanup
| * | | input_engine: Fix typo in TriggerOnAxisChange() parameter nameGravatar Lioncash2021-12-131-1/+1
| | | |
| * | | input_engine: Simplify PreSet* family of functionsGravatar Lioncash2021-12-132-24/+14
| | | | | | | | | | | | | | | | | | | | We can make use of try_emplace() to insert values only if they don't already exist.
| * | | input_engine: Avoid redundant map lookupsGravatar Lioncash2021-12-131-16/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We can use iterators to avoid looking up into maps twice in the getter functions. At the same time we can also avoid copying the ControllerData structs, since they're 264 bytes in size.
| * | | input_engine: Remove left-over namespace qualifiersGravatar Lioncash2021-12-131-3/+3
| | | | | | | | | | | | | | | | These types are part of the InputCommon namespace.
| * | | input_engine: Iterate by reference rather than by value where applicableGravatar Lioncash2021-12-131-10/+10
| | | | | | | | | | | | | | | | | | | | Avoids creating copies of several object instances (some of which being over 100 bytes in size).
| * | | input_engine: Take BasicMotion by const reference with SetMotion() and ↵Gravatar Lioncash2021-12-133-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | TriggerOnMotionChange() Copies the BasicMotion instance once instead of twice.
| * | | input_engine: std::move InputIdentifier in SetCallback()Gravatar Lioncash2021-12-131-1/+1
| | | | | | | | | | | | | | | | Allows avoiding std::function allocations.
| * | | input_engine: Pass LedStatus by const referenceGravatar Lioncash2021-12-132-2/+2
| | | | | | | | | | | | | | | | Avoids copies where reasonably applicable
| * | | input_engine: Pass VibrationStatus by const reference in SetRumble()Gravatar Lioncash2021-12-136-8/+10
| | | | | | | | | | | | | | | | Avoids creating copies of the struct where not necessary.
| * | | input_engine: std::move engine name where applicableGravatar Lioncash2021-12-1315-29/+29
| | | | | | | | | | | | | | | | | | | | We can allow the name to be moved into, allowing allocations to be avoided.
| * | | input_engine: Remove callback clearing in constructorGravatar Lioncash2021-12-131-3/+1
| | | | | | | | | | | | | | | | | | | | The callback map is a member variable, so this will always be empty on initial construction.
| * | | input_engine: Remove unnecessary semi-colonsGravatar Lioncash2021-12-131-6/+6
| | | | | | | | | | | | | | | | Silences -Wextra-semi warnings
| * | | input_engine: Remove unnecessary returnGravatar Lioncash2021-12-131-3/+1
| |/ / | | | | | | | | | This is a void function, so it doesn't need this.
* | | tas_input: Avoid minor copies in Read/WriteCommandButtons()Gravatar Lioncash2021-12-131-2/+2
| | | | | | | | | | | | We don't need to copy the whole pair
* | | tas_input: Remove unnecessary semicolonGravatar Lioncash2021-12-131-1/+1
| | | | | | | | | | | | Resolves a -Wextra-semi warning
* | | tas_input: Execute clear() even if emptyGravatar Lioncash2021-12-131-3/+2
| | | | | | | | | | | | | | | clear() when empty is simply a no-op, so we can get rid of the check here and let the stdlib do it for us.
* | | tas_input: Remove unnecessary includesGravatar Lioncash2021-12-131-2/+2
| | | | | | | | | | | | | | | Gets rid of indirect includes and includes only what the interface needs.
* | | tas_input: std::move strings into vectorGravatar Lioncash2021-12-131-21/+24
| | | | | | | | | | | | | | | While we're in the same area, we can also avoid performing std::stoi in a loop when it only needs to be performed once.
* | | tas_input: Use istringstream over stringstreamGravatar Lioncash2021-12-131-2/+2
| | | | | | | | | | | | | | | This is only using the input facilities, so we don't need to use the fully-fleged stringstream.
* | | tas_input: Use u8string_view instead of u8stringGravatar Lioncash2021-12-132-6/+7
| | | | | | | | | | | | Same behavior, but without the potential for extra allocations.
* | | tas_input: Remove unused std::smatch variableGravatar Lioncash2021-12-131-2/+0
| | | | | | | | | | | | This also means we can get rid of the dependency on <regex>
HIC SVNT DRACONES