RunnableTask/FunctionTask run(): Write tExecuted in finally block, removing code redundancy and placing write at end of operation.

1 files changed, 3 insertions, 4 deletions

diff --git a/src/java/com/jogamp/common/util/RunnableTask.java b/src/java/com/jogamp/common/util/RunnableTask.java
index ce4688d..c18556b 100644
--- a/src/java/com/jogamp/common/util/RunnableTask.java
+++ b/src/java/com/jogamp/common/util/RunnableTask.java
@@ -93,9 +93,7 @@ public class RunnableTask extends TaskBase {
         if(null == syncObject) {
             try {
                 runnable.run();
-                tExecuted = System.currentTimeMillis();
             } catch (Throwable t) {
-                tExecuted = System.currentTimeMillis();
                 runnableException = t;
                 if(null != exceptionOut) {
                     exceptionOut.println("RunnableTask.run(): "+getExceptionOutIntro()+" exception occured on thread "+Thread.currentThread().getName()+": "+toString());
@@ -105,14 +103,14 @@ public class RunnableTask extends TaskBase {
                 if(!catchExceptions) {
                     throw new RuntimeException(runnableException);
                 }
+            } finally {
+                tExecuted = System.currentTimeMillis();                
             }
         } else {
             synchronized (syncObject) {
                 try {
                     runnable.run();
-                    tExecuted = System.currentTimeMillis();
                 } catch (Throwable t) {
-                    tExecuted = System.currentTimeMillis();
                     runnableException = t;
                     if(null != exceptionOut) {
                         exceptionOut.println("RunnableTask.run(): "+getExceptionOutIntro()+" exception occured on thread "+Thread.currentThread().getName()+": "+toString());
@@ -123,6 +121,7 @@ public class RunnableTask extends TaskBase {
                         throw new RuntimeException(runnableException);
                     }
                 } finally {
+                    tExecuted = System.currentTimeMillis();
                     syncObject.notifyAll();
                 }
             }