Clean/Fix: Threading Code

- Remove unsafe double checked locking
- Annotate safe double checked locking (volatile)
- use 'static final' if possible

1 files changed, 4 insertions, 3 deletions

diff --git a/src/jogl/classes/jogamp/opengl/GLWorkerThread.java b/src/jogl/classes/jogamp/opengl/GLWorkerThread.java
index e57cbe0bc..ac9655fbb 100644
--- a/src/jogl/classes/jogamp/opengl/GLWorkerThread.java
+++ b/src/jogl/classes/jogamp/opengl/GLWorkerThread.java
@@ -40,7 +40,6 @@
 package jogamp.opengl;
 
 import java.lang.reflect.InvocationTargetException;
-import java.security.*;
 import java.util.*;
 import javax.media.opengl.*;
 
@@ -69,9 +68,11 @@ public class GLWorkerThread {
   
   /** Should only be called by Threading class if creation of the
       GLWorkerThread was requested via the opengl.1thread system
-      property. */
+      property. <br>
+   * Start the GLWorkerThread iff not started yet!
+   */
   public static void start() {
-    if (!started) {
+    if (!started) { // volatile: ok
       synchronized (GLWorkerThread.class) {
         if (!started) {
           lock = new Object();