From db25b1ba6575210741e485838d0882a1590125e6 Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Tue, 8 Jul 2014 10:33:44 +0200 Subject: Findbugs.synchronization: Fix double-check w/o volatile; Remove redundant synchronized from setter --- src/jogl/classes/com/jogamp/opengl/util/FPSAnimator.java | 2 +- src/nativewindow/classes/jogamp/nativewindow/windows/GDIUtil.java | 2 +- 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()} -- cgit v1.2.3