Controls: Make the analog/digital mapping clash resolution more gentle.

Now takes the time into account, so clashing digital input will only shrink
analog inputs once it's a few seconds old.

Also fixes a bug where if there are both inputs, it was hard to reach
the limits because the digital input itself ended up getting shrunk.

This might help #17860

Common/Input/InputState.h

@@ -36,6 +36,7 @@ enum InputDeviceID {
 	DEVICE_ID_XR_CONTROLLER_LEFT = 40,
 	DEVICE_ID_XR_CONTROLLER_RIGHT = 41,
 	DEVICE_ID_TOUCH = 42,
+	DEVICE_ID_COUNT,
 };
 
 inline InputDeviceID operator +(InputDeviceID deviceID, int addend) {
@@ -121,6 +122,9 @@ public:
 		if (keyCode != other.keyCode) return false;
 		return true;
 	}
+	bool operator != (const InputMapping &other) const {
+		return !(*this == other);
+	}
 
 	void FormatDebug(char *buffer, size_t bufSize) const;
 };

Core/ControlMapper.cpp

@@ -161,7 +161,7 @@ void ControlMapper::ForceReleaseVKey(int vkey) {
 			for (const auto &mapping : entry.mappings) {
 				curInput_[mapping] = { 0.0f, now };
 				// Different logic for signed axes?
-				UpdatePSPState(mapping);
+				UpdatePSPState(mapping, now);
 			}
 		}
 	}
@@ -179,8 +179,11 @@ static int RotatePSPKeyCode(int x) {
 }
 
 // Used to decay analog values when clashing with digital ones.
-static ControlMapper::InputSample ReduceMagnitude(ControlMapper::InputSample sample) {
-	sample.value *= 0.75f;
+static ControlMapper::InputSample ReduceMagnitude(ControlMapper::InputSample sample, double now) {
+	float reduction = std::min(std::max(0.0f, (float)(now - sample.timestamp) - 2.0f), 1.0f);
+	if (reduction > 0.0f) {
+		sample.value *= (1.0f - reduction);
+	}
 	if ((sample.value > 0.0f && sample.value < 0.05f) || (sample.value < 0.0f && sample.value > -0.05f)) {
 		sample.value = 0.0f;
 	}
@@ -248,7 +251,8 @@ void ControlMapper::SwapMappingIfEnabled(uint32_t *vkey) {
 }
 
 // Can only be called from Key or Axis.
-bool ControlMapper::UpdatePSPState(const InputMapping &changedMapping) {
+// TODO: We should probably make a batched version of this.
+bool ControlMapper::UpdatePSPState(const InputMapping &changedMapping, double now) {
 	// Instead of taking an input key and finding what it outputs, we loop through the OUTPUTS and
 	// see if the input that corresponds to it has a value. That way we can easily implement all sorts
 	// of crazy input combos if needed.
@@ -363,7 +367,11 @@ bool ControlMapper::UpdatePSPState(const InputMapping &changedMapping) {
 		if (!changedMapping.IsAxis()) {
 			for (auto &multiMapping : inputMappings) {
 				for (auto &mapping : multiMapping.mappings) {
-					curInput_[mapping] = ReduceMagnitude(curInput_[mapping]);
+					if (mapping != changedMapping && curInput_[mapping].value > 0.0f) {
+						// Note that this takes the time into account now - values will
+						// decay after a while, not immediately.
+						curInput_[mapping] = ReduceMagnitude(curInput_[mapping], now);
+					}
 				}
 			}
 		}
@@ -414,15 +422,19 @@ bool ControlMapper::Key(const KeyInput &key, bool *pauseTrigger) {
 		// Claim that we handled this. Prevents volume key repeats from popping up the volume control on Android.
 		return true;
 	}
-
-	std::lock_guard<std::mutex> guard(mutex_);
+	double now = time_now_d();
+	if (key.deviceId < DEVICE_ID_COUNT) {
+		deviceTimestamps_[(int)key.deviceId] = now;
+	}
 
 	InputMapping mapping(key.deviceId, key.keyCode);
 
+	std::lock_guard<std::mutex> guard(mutex_);
+
 	if (key.flags & KEY_DOWN) {
-		curInput_[mapping] = { 1.0f, time_now_d() };
+		curInput_[mapping] = { 1.0f, now };
 	} else if (key.flags & KEY_UP) {
-		curInput_[mapping] = { 0.0f, time_now_d() };
+		curInput_[mapping] = { 0.0f, now};
 	}
 
 	// TODO: See if this can be simplified further somehow.
@@ -435,7 +447,7 @@ bool ControlMapper::Key(const KeyInput &key, bool *pauseTrigger) {
 		}
 	}
 
-	return UpdatePSPState(mapping);
+	return UpdatePSPState(mapping, now);
 }
 
 void ControlMapper::ToggleSwapAxes() {
@@ -464,35 +476,37 @@ void ControlMapper::ToggleSwapAxes() {
 }
 
 void ControlMapper::Axis(const AxisInput &axis) {
-	std::lock_guard<std::mutex> guard(mutex_);
 	double now = time_now_d();
+
+	std::lock_guard<std::mutex> guard(mutex_);
+	if (axis.deviceId < DEVICE_ID_COUNT) {
+		deviceTimestamps_[(int)axis.deviceId] = now;
+	}
 	if (axis.value >= 0.0f) {
 		InputMapping mapping(axis.deviceId, axis.axisId, 1);
 		InputMapping opposite(axis.deviceId, axis.axisId, -1);
 		curInput_[mapping] = { axis.value, now };
 		curInput_[opposite] = { 0.0f, now };
-		UpdatePSPState(mapping);
-		UpdatePSPState(opposite);
+		UpdatePSPState(mapping, now);
+		UpdatePSPState(opposite, now);
 	} else if (axis.value < 0.0f) {
 		InputMapping mapping(axis.deviceId, axis.axisId, -1);
 		InputMapping opposite(axis.deviceId, axis.axisId, 1);
 		curInput_[mapping] = { -axis.value, now };
 		curInput_[opposite] = { 0.0f, now };
-		UpdatePSPState(mapping);
-		UpdatePSPState(opposite);
+		UpdatePSPState(mapping, now);
+		UpdatePSPState(opposite, now);
 	}
 }
 
-void ControlMapper::Update() {
+void ControlMapper::Update(double now) {
 	if (autoRotatingAnalogCW_) {
-		const double now = time_now_d();
 		// Clamp to a square
 		float x = std::min(1.0f, std::max(-1.0f, 1.42f * (float)cos(now * -g_Config.fAnalogAutoRotSpeed)));
 		float y = std::min(1.0f, std::max(-1.0f, 1.42f * (float)sin(now * -g_Config.fAnalogAutoRotSpeed)));
 
 		setPSPAnalog_(0, x, y);
 	} else if (autoRotatingAnalogCCW_) {
-		const double now = time_now_d();
 		float x = std::min(1.0f, std::max(-1.0f, 1.42f * (float)cos(now * g_Config.fAnalogAutoRotSpeed)));
 		float y = std::min(1.0f, std::max(-1.0f, 1.42f * (float)sin(now * g_Config.fAnalogAutoRotSpeed)));
 

Core/ControlMapper.h

@@ -11,7 +11,7 @@
 // Main use is of course from EmuScreen.cpp, but also useful from control settings etc.
 class ControlMapper {
 public:
-	void Update();
+	void Update(double now);
 
 	// Inputs to the table-based mapping
 	// These functions are free-threaded.
@@ -48,7 +48,7 @@ public:
 	};
 
 private:
-	bool UpdatePSPState(const InputMapping &changedMapping);
+	bool UpdatePSPState(const InputMapping &changedMapping, double now);
 	float MapAxisValue(float value, int vkId, const InputMapping &mapping, const InputMapping &changedMapping, bool *oppositeTouched);
 	void SwapMappingIfEnabled(uint32_t *vkey);
 
@@ -62,6 +62,8 @@ private:
 	float virtKeys_[VIRTKEY_COUNT]{};
 	bool virtKeyOn_[VIRTKEY_COUNT]{};  // Track boolean output separaately since thresholds may differ.
 
+	double deviceTimestamps_[42]{};
+
 	int lastNonDeadzoneDeviceID_[2]{};
 
 	float history_[2][2]{};

UI/ControlMappingScreen.cpp

@@ -512,7 +512,7 @@ AnalogSetupScreen::AnalogSetupScreen(const Path &gamePath) : UIDialogScreenWithG
 }
 
 void AnalogSetupScreen::update() {
-	mapper_.Update();
+	mapper_.Update(time_now_d());
 	// We ignore the secondary stick for now and just use the two views
 	// for raw and psp input.
 	if (stickView_[0]) {

UI/EmuScreen.cpp

@@ -1117,7 +1117,9 @@ void EmuScreen::update() {
 	if (invalid_)
 		return;
 
-	controlMapper_.Update();
+	double now = time_now_d();
+
+	controlMapper_.Update(now);
 
 	if (pauseTrigger_) {
 		pauseTrigger_ = false;
@@ -1137,7 +1139,7 @@ void EmuScreen::update() {
 			saveStatePreview_->SetFilename(fn);
 			if (!fn.empty()) {
 				saveStatePreview_->SetVisibility(UI::V_VISIBLE);
-				saveStatePreviewShownTime_ = time_now_d();
+				saveStatePreviewShownTime_ = now;
 			} else {
 				saveStatePreview_->SetVisibility(UI::V_GONE);
 			}
@@ -1145,10 +1147,10 @@ void EmuScreen::update() {
 
 		if (saveStatePreview_->GetVisibility() == UI::V_VISIBLE) {
 			double endTime = saveStatePreviewShownTime_ + 2.0;
-			float alpha = clamp_value((endTime - time_now_d()) * 4.0, 0.0, 1.0);
+			float alpha = clamp_value((endTime - now) * 4.0, 0.0, 1.0);
 			saveStatePreview_->SetColor(colorAlpha(0x00FFFFFF, alpha));
 
-			if (time_now_d() - saveStatePreviewShownTime_ > 2) {
+			if (now - saveStatePreviewShownTime_ > 2) {
 				saveStatePreview_->SetVisibility(UI::V_GONE);
 			}
 		}