Merge pull request #1570 from matthiasblaesing/cleanup

Integrate reported quality issues

Author: matthiasblaesing

src/com/sun/jna/Function.java

@@ -245,7 +245,7 @@ public static Function getFunction(Pointer p, int callFlags, String encoding) {
         this.library = library;
         this.functionName = functionName;
         this.callFlags = callFlags;
-        this.options = library.options;
+        this.options = library.getOptions();
         this.encoding = encoding != null ? encoding : Native.getDefaultStringEncoding();
         try {
             this.peer = library.getSymbolAddress(functionName);

src/com/sun/jna/NativeLibrary.java

@@ -95,15 +95,15 @@ public long getSymbolAddress(long handle, String name, SymbolProvider parent) {
         }
     };
 
-    private Cleaner.Cleanable cleanable;
-    private long handle;
+    private final Cleaner.Cleanable cleanable;
+    private volatile long handle;
     private final String libraryName;
     private final String libraryPath;
     private final Map<String, Function> functions = new HashMap<>();
     private final SymbolProvider symbolProvider;
-    final int callFlags;
-    private String encoding;
-    final Map<String, ?> options;
+    private final int callFlags;
+    private final String encoding;
+    private final Map<String, ?> options;
 
     private static final Map<String, Reference<NativeLibrary>> libraries = new HashMap<>();
 
@@ -120,6 +120,7 @@ private static String functionKey(String name, int flags, String encoding) {
         return name + "|" + flags + "|" + encoding;
     }
 
+    @SuppressWarnings("LeakingThisInConstructor")
     private NativeLibrary(String libraryName, String libraryPath, long handle, Map<String, ?> options) {
         this.libraryName = getLibraryName(libraryName);
         this.libraryPath = libraryPath;
@@ -129,17 +130,18 @@ private NativeLibrary(String libraryName, String libraryPath, long handle, Map<S
         int callingConvention = option instanceof Number ? ((Number)option).intValue() : Function.C_CONVENTION;
         this.callFlags = callingConvention;
         this.options = options;
-        this.encoding = (String)options.get(Library.OPTION_STRING_ENCODING);
         SymbolProvider optionSymbolProvider = (SymbolProvider)options.get(Library.OPTION_SYMBOL_PROVIDER);
         if (optionSymbolProvider == null) {
             this.symbolProvider = NATIVE_SYMBOL_PROVIDER;
         } else {
             this.symbolProvider = optionSymbolProvider;
         }
 
-        if (this.encoding == null) {
-            this.encoding = Native.getDefaultStringEncoding();
+        String encodingValue = (String) options.get(Library.OPTION_STRING_ENCODING);
+        if (encodingValue == null) {
+            encodingValue = Native.getDefaultStringEncoding();
         }
+        this.encoding = encodingValue;
 
         // Special workaround for w32 kernel32.GetLastError
         // Short-circuit the function to use built-in GetLastError access
@@ -699,8 +701,8 @@ public void close() {
 
         synchronized(this) {
             if (handle != 0) {
-                cleanable.clean();
                 handle = 0;
+                cleanable.clean();
             }
         }
     }