From 9c3a7e290491d1c64c6d94a8651a8f13918d0b4e Mon Sep 17 00:00:00 2001
From: Charlie Groves <charlie@threerings.net>
Date: Thu, 6 Mar 2008 01:24:21 +0000
Subject: [PATCH] Calibrate every 5 seconds instead of every 1000 calls.  When
 many things are being animated, the timer will get a ton of calls and the
 variation in the number of millis elapsed per timer time may be leading to
 unwanted variations in the drift value.

git-svn-id: svn+ssh://src.earth.threerings.net/nenya/trunk@432 ed5b42cb-e716-0410-a449-f6a68f950b19
---
 .../com/threerings/media/timer/CalibratingTimer.java  | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/java/com/threerings/media/timer/CalibratingTimer.java b/src/java/com/threerings/media/timer/CalibratingTimer.java
index 6ee2ad3d..1325bfb0 100644
--- a/src/java/com/threerings/media/timer/CalibratingTimer.java
+++ b/src/java/com/threerings/media/timer/CalibratingTimer.java
@@ -19,6 +19,7 @@ public abstract class CalibratingTimer
     protected void init (long milliDivider, long microDivider)
     {
         _milliDivider = milliDivider;
+        _calibrationInterval = 5 * 1000 * _milliDivider;// Calibrate every 5 seconds
         _microDivider = microDivider;
         reset();
         Log.info("Using " + getClass() + " timer [mfreq=" + _milliDivider + ", ufreq="
@@ -43,16 +44,15 @@ public abstract class CalibratingTimer
         _startStamp = _priorCurrent = current();
         _driftMilliStamp = System.currentTimeMillis();
         _driftTimerStamp = current();
-        _callsSinceCalibration = 0;
     }
 
     /** Returns the difference between _startStamp and current() */
     protected long elapsed ()
     {
-        if (_callsSinceCalibration++ > 1000) {
+        long current = current();
+        if ((current - _driftTimerStamp) > _calibrationInterval) {
             calibrate();
         }
-        long current = current();
         if (_driftRatio != 1.0) {
             long elapsed = current - _priorCurrent;
             _startStamp += (elapsed - (elapsed * _driftRatio));
@@ -77,7 +77,6 @@ public abstract class CalibratingTimer
         }
         _driftMilliStamp = System.currentTimeMillis();
         _driftTimerStamp = current();
-        _callsSinceCalibration = 0;
     }
     
     /** Return the current value for this timer. */
@@ -104,7 +103,7 @@ public abstract class CalibratingTimer
     /** Ratio of currentTimeMillis to timer millis. */
     protected double _driftRatio = 1.0;
 
-    /** Number of calls to elapsed since we've called calibrate. */
-    protected int _callsSinceCalibration = 0;
+    /** Amount of timer ticks that elapse between calibrations. */
+    protected long _calibrationInterval;
 
 }