From f0e9a844fccebf295550daa2a0a41eb54a9f9f08 Mon Sep 17 00:00:00 2001
From: asofold <asofold@web.de>
Date: Sun, 13 Jan 2013 22:11:29 +0100
Subject: [PATCH] ActionFrequency.update: reorder safety checks slightly.

Gains a little bit of consistency for negative time differences. Also
might be faster for the case of seldom shifting, though most likely
by an insignificant amount.
---
 .../nocheatplus/utilities/ActionFrequency.java            | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/utilities/ActionFrequency.java b/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/utilities/ActionFrequency.java
index 6750b986..ba83d181 100644
--- a/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/utilities/ActionFrequency.java
+++ b/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/utilities/ActionFrequency.java
@@ -50,16 +50,16 @@ public class ActionFrequency {
 	 */
 	public final void update(final long now) {
 		final long diff = now - time;
-		final int shift = (int) ((float) diff / (float) durBucket);
-		if (shift == 0){
-			// No update.
+		if (diff < durBucket){
+			// No update (first bucket).
 			return; 
 		}
-		else if (shift >= buckets.length || shift < 0){
+		else if (diff >= durBucket * buckets.length || diff < 0){
 			// Clear (beyond range).
 			clear(now);
 			return;
 		}
+		final int shift = (int) ((float) diff / (float) durBucket);
 		// Update buckets.
 		for (int i = 0; i < buckets.length - shift; i++){
 			buckets[buckets.length - (i + 1)] = buckets[buckets.length - (i + 1 + shift)];