refactor: multiply float only if sensitivity is adjusted

Apply scaling only when necessary as it has a performance cost
Xtr126 · Aug 24, 2024 · ae3904d · ae3904d
1 parent 53b58c0
commit ae3904d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/app/src/main/java/xtr/keymapper/mouse/MouseAimConfig.java b/app/src/main/java/xtr/keymapper/mouse/MouseAimConfig.java
@@ -68,7 +68,7 @@ public String getData() {
  + width + " " + height + " "
  + xleftClick + " " + yleftClick + " "
  + xSensitivity + " " + ySensitivity + " "
- + (limitedBounds ? 1 : 0);
+ + (applyNonLinearScaling ? 1 : 0);
  }
 
  public void setCenterXY(MovableFrameLayout crosshair){

diff --git a/app/src/main/java/xtr/keymapper/mouse/MouseAimHandler.java b/app/src/main/java/xtr/keymapper/mouse/MouseAimHandler.java
@@ -106,13 +106,19 @@ public float calculateScaledX(int value) {
  } else {
  return value * config.xSensitivity;
  }
- } else {
+ } else if (config.xSensitivity != 1.0) {
  return value * config.xSensitivity;
+ } else {
+ return value;
  }
  }
 
  private float calculateScaledY(int value) {
- return value * config.ySensitivity;
+ if (config.ySensitivity != 1.0) {
+ return value * config.ySensitivity;
+ } else {
+ return value;
+ }
  }
 
  public void stop() {