[piccolo2d-dev] [piccolo2d] r723 committed - Fixing unfortunately infinite loop caused by wrapping package scoped "...

[codesite-noreply]

Revision: 723
Author: allain.lalonde
Date: Wed Oct 14 19:17:06 2009
Log: Fixing unfortunately infinite loop caused by wrapping package  
scoped "running" variable in a method.
http://code.google.com/p/piccolo2d/source/detail?r=723

Modified:
   
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimer.java
   
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimerQueue.java

=======================================
---  
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimer.java   
 
Wed Oct 14 14:23:20 2009
+++  
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimer.java   
 
Wed Oct 14 19:17:06 2009
@@ -57,7 +57,7 @@
      // happen in applet case when TimerQueues thread is destroyed.
      private long expirationTime;
      private SWTTimer nextTimer;
-    private boolean running;
+    boolean running;

      /**
       * DoPostEvent is a runnable class that fires actionEvents to the  
listeners
=======================================
---  
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimerQueue.java
       
Wed Oct 14 14:23:20 2009
+++  
/piccolo2d.java/trunk/swt/src/main/java/edu/umd/cs/piccolox/swt/SWTTimerQueue.java
       
Wed Oct 14 19:17:06 2009
@@ -213,7 +213,8 @@
      }

      synchronized boolean containsTimer(final SWTTimer timer) {
-        return timer.isRunning();
+        //TODO: making this use isRunning without causing an infinite loop
+        return timer.running;
      }

      /**

--~--~---------~--~----~------------~-------~--~----~
Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
-~----------~----~----~----~------~----~------~--~---