From df9ff7f340a5ab4e07efc613f5f264eeae63d4c7 Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Thu, 3 Jul 2014 16:06:47 +0200 Subject: Code Clean-Up based on our Recommended Settings (jogamp-scripting c47bc86ae2ee268a1f38c5580d11f93d7f8d6e74) Code Clean-Up based on our Recommended Settings (jogamp-scripting c47bc86ae2ee268a1f38c5580d11f93d7f8d6e74) - Change non static accesses to static members using declaring type - Change indirect accesses to static members to direct accesses (accesses through subtypes) - Add final modifier to private fields - Add final modifier to method parameters - Add final modifier to local variables - Remove unnecessary casts - Remove unnecessary '$NON-NLS$' tags - Remove trailing white spaces on all lines --- src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java') diff --git a/src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java b/src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java index 76bb2f8..de69d0c 100644 --- a/src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java +++ b/src/java/jogamp/common/os/WindowsDynamicLinkerImpl.java @@ -44,27 +44,27 @@ public final class WindowsDynamicLinkerImpl extends DynamicLinkerImpl { private static native long LoadLibraryW(java.lang.String lpLibFileName); @Override - public final long openLibraryLocal(String libraryName, boolean debug) throws SecurityException { + public final long openLibraryLocal(final String libraryName, final boolean debug) throws SecurityException { // How does that work under Windows ? // Don't know .. so it's an alias for the time being return openLibraryGlobal(libraryName, debug); } @Override - public final long openLibraryGlobal(String libraryName, boolean debug) throws SecurityException { + public final long openLibraryGlobal(final String libraryName, final boolean debug) throws SecurityException { SecurityUtil.checkLinkPermission(libraryName); final long handle = LoadLibraryW(libraryName); if( 0 != handle ) { incrLibRefCount(handle, libraryName); } else if ( DEBUG || debug ) { - int err = GetLastError(); + final int err = GetLastError(); System.err.println("LoadLibraryW \""+libraryName+"\" failed, error code: 0x"+Integer.toHexString(err)+", "+err); } return handle; } @Override - public final long lookupSymbolGlobal(String symbolName) throws SecurityException { + public final long lookupSymbolGlobal(final String symbolName) throws SecurityException { SecurityUtil.checkAllLinkPermission(); if(DEBUG_LOOKUP) { System.err.println("lookupSymbolGlobal: Not supported on Windows"); @@ -74,7 +74,7 @@ public final class WindowsDynamicLinkerImpl extends DynamicLinkerImpl { } @Override - public final long lookupSymbol(long libraryHandle, String symbolName) throws IllegalArgumentException { + public final long lookupSymbol(final long libraryHandle, final String symbolName) throws IllegalArgumentException { if( null == getLibRef( libraryHandle ) ) { throw new IllegalArgumentException("Library handle 0x"+Long.toHexString(libraryHandle)+" unknown."); } @@ -97,7 +97,7 @@ public final class WindowsDynamicLinkerImpl extends DynamicLinkerImpl { } @Override - public final void closeLibrary(long libraryHandle) throws IllegalArgumentException { + public final void closeLibrary(final long libraryHandle) throws IllegalArgumentException { if( null == decrLibRefCount( libraryHandle ) ) { throw new IllegalArgumentException("Library handle 0x"+Long.toHexString(libraryHandle)+" unknown."); } -- cgit v1.2.3