summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSven Gothel <[email protected]>2014-07-08 10:33:44 +0200
committerSven Gothel <[email protected]>2014-07-08 10:33:44 +0200
commitdb25b1ba6575210741e485838d0882a1590125e6 (patch)
treedf32d588d1edcaff2606694776f1299a70acd621
parentb5910f18f0b82a8a1f6f6252dc19971d5e487f39 (diff)
Findbugs.synchronization: Fix double-check w/o volatile; Remove redundant synchronized from setter
-rw-r--r--src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java2
-rw-r--r--src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java b/src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java
index de99ae70d..746b642c2 100644
--- a/src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java
+++ b/src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java
@@ -106,7 +106,7 @@ public class FPSAnimator extends AnimatorBase {
* @param fps
* @throws GLException if the animator has already been started
*/
- public final synchronized void setFPS(final int fps) throws GLException {
+ public final void setFPS(final int fps) throws GLException {
if ( isStarted() ) {
throw new GLException("Animator already started.");
}
diff --git a/src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java b/src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java
index 8203a804b..31e85dcee 100644
--- a/src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java
+++ b/src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java
@@ -40,7 +40,7 @@ public class GDIUtil implements ToolkitProperties {
private static final String dummyWindowClassNameBase = "_dummyWindow_clazz" ;
private static RegisteredClassFactory dummyWindowClassFactory;
- private static boolean isInit = false;
+ private static volatile boolean isInit = false;
/**
* Called by {@link NativeWindowFactory#initSingleton()}