Attributes: Allow detecting if attributes are available at all.

NCPCompatBukkit/src/main/java/fr/neatmonster/nocheatplus/compat/bukkit/MCAccessBukkitBase.java

@@ -231,12 +231,12 @@ public class MCAccessBukkitBase implements MCAccess {
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.30000002 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2512/src/main/java/fr/neatmonster/nocheatplus/compat/cb2512/MCAccessCB2512.java

@@ -114,12 +114,12 @@ public class MCAccessCB2512 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2545/src/main/java/fr/neatmonster/nocheatplus/compat/cb2545/MCAccessCB2545.java

@@ -114,12 +114,12 @@ public class MCAccessCB2545 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2602/src/main/java/fr/neatmonster/nocheatplus/compat/cb2602/MCAccessCB2602.java

@@ -115,12 +115,12 @@ public class MCAccessCB2602 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2645/src/main/java/fr/neatmonster/nocheatplus/compat/cb2645/MCAccessCB2645.java

@@ -116,12 +116,12 @@ public class MCAccessCB2645 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2691/src/main/java/fr/neatmonster/nocheatplus/compat/cb2691/MCAccessCB2691.java

@@ -118,12 +118,12 @@ public class MCAccessCB2691 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCompatCB2763/src/main/java/fr/neatmonster/nocheatplus/compat/cb2763/MCAccessCB2763.java

@@ -116,12 +116,12 @@ public class MCAccessCB2763 implements MCAccess{
 
     @Override
     public double getSpeedAttributeMultiplier(Player player) {
-        return 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override
     public double getSprintAttributeMultiplier(Player player) {
-        return player.isSprinting() ? 1.3 : 1.0;
+        return Double.MAX_VALUE;
     }
 
     @Override

NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/MovingListener.java

@@ -438,8 +438,11 @@ public class MovingListener extends CheckListener implements TickListener, IRemo
             if (player.getFoodLevel() > 5) {
                 data.timeSprinting = time;
                 data.multSprinting = mcAccess.getSprintAttributeMultiplier(player);
-                // Server side can be inconsistent, so the multiplier might be plain wrong (1.0).
-                if (cc.assumeSprint && data.multSprinting == 1.0) {
+                if (data.multSprinting == Double.MAX_VALUE) {
+                    data.multSprinting = 1.30000002;
+                }
+                else if (cc.assumeSprint && data.multSprinting == 1.0) {
+                    // Server side can be inconsistent, so the multiplier might be plain wrong (1.0).
                     // TODO: Could be more/less than actual, but "infinite" latency would not work either.
                     data.multSprinting = 1.30000002;
                 }

NCPCore/src/main/java/fr/neatmonster/nocheatplus/checks/moving/SurvivalFly.java

@@ -654,7 +654,10 @@ public class SurvivalFly extends Check {
         }
 
         // Attributes in here.
-        hAllowedDistance *= mcAccess.getSpeedAttributeMultiplier(player);
+        final double attrMod = mcAccess.getSpeedAttributeMultiplier(player);
+        if (attrMod != Double.MAX_VALUE) {
+            hAllowedDistance *= attrMod;
+        }
 
         // Short cut.
         // TODO: Check if a) early return makes sense and b) do it for each of the following parts.

NCPCore/src/main/java/fr/neatmonster/nocheatplus/compat/MCAccess.java

@@ -93,17 +93,21 @@ public interface MCAccess {
     public double getFasterMovementAmplifier(Player player);
 
     /**
+     * Generic speed modifier as a multiplier.
      * 
      * @param player
      * @return A multiplier for the allowed speed, excluding the sprint boost
-     *         modifier (!). If not possible to determine, it should be 1.0.
+     *         modifier (!). If not possible to determine, it should
+     *         Double.MAX_VALUE.
      */
     public double getSpeedAttributeMultiplier(Player player);
 
     /**
-     * Modifier for sprinting.
+     * Sprint boost modifier as a multiplier.
+     * 
      * @param player
-     * @return
+     * @return The sprint boost modifier as a multiplier. If not possible to
+     *         determine, it should be Double.MAX_VALUE.
      */
     public double getSprintAttributeMultiplier(Player player);