Extracted native Windows logic and fixed #194

src/main/java/net/sf/jabref/JabRef.java

@@ -33,6 +33,7 @@
 import javax.swing.plaf.FontUIResource;
 
 import net.sf.jabref.gui.*;
+import net.sf.jabref.gui.native_ext.WindowsExtensions;
 import net.sf.jabref.importer.fetcher.EntryFetcher;
 import net.sf.jabref.importer.fetcher.EntryFetchers;
 import net.sf.jabref.logic.journals.Abbreviations;
@@ -62,10 +63,6 @@
 import net.sf.jabref.wizard.auximport.AuxCommandLine;
 
 import com.sun.jna.Native;
-import com.sun.jna.NativeLong;
-import com.sun.jna.Pointer;
-import com.sun.jna.WString;
-import com.sun.jna.ptr.PointerByReference;
 
 /**
  * JabRef Main Class - The application gets started here.
@@ -150,10 +147,8 @@ public void start(String[] args) {
         Globals.NEWLINE = Globals.prefs.get(JabRefPreferences.NEWLINE);
 
         if (OS.WINDOWS) {
-            // Set application user model id so that pinning JabRef to the Win7/8 taskbar works
-            // Based on http://stackoverflow.com/a/1928830
-            JabRef.setCurrentProcessExplicitAppUserModelID("JabRef." + Globals.BUILD_INFO.getVersion());
-            //System.out.println(getCurrentProcessExplicitAppUserModelID());
+            // activate pin to taskbar for Windows 7 and up
+            WindowsExtensions.enablePinToTaskbar();
         }
 
         Vector<ParserResult> loaded = processArguments(args, true);
@@ -165,46 +160,12 @@ public void start(String[] args) {
 
         openWindow(loaded);
     }
-    
+
     private void setupLogHandlerForErrorConsole() {
         Globals.handler = new CacheableHandler();
         ((Jdk14Logger)LOGGER).getLogger().addHandler(Globals.handler);
     }
 
-    // Do not use this code in release version, it contains some memory leaks
-    public static String getCurrentProcessExplicitAppUserModelID()
-    {
-        final PointerByReference r = new PointerByReference();
-
-        if (JabRef.GetCurrentProcessExplicitAppUserModelID(r).longValue() == 0)
-        {
-            final Pointer p = r.getValue();
-
-            return p.getString(0, true); // here we leak native memory by lazyness
-        }
-        return "N/A";
-    }
-
-    private static void setCurrentProcessExplicitAppUserModelID(final String appID)
-    {
-        if (JabRef.SetCurrentProcessExplicitAppUserModelID(new WString(appID)).longValue() != 0) {
-            throw new RuntimeException("unable to set current process explicit AppUserModelID to: " + appID);
-        }
-    }
-
-    private static native NativeLong GetCurrentProcessExplicitAppUserModelID(PointerByReference appID);
-
-    private static native NativeLong SetCurrentProcessExplicitAppUserModelID(WString appID);
-
-
-    static
-    {
-        if (OS.WINDOWS) {
-            Native.register("shell32");
-        }
-    }
-
-
     public Vector<ParserResult> processArguments(String[] args, boolean initialStartup) {
 
         cli = new JabRefCLI(args);

src/main/java/net/sf/jabref/gui/native_ext/WindowsExtensions.java

@@ -0,0 +1,70 @@
+package net.sf.jabref.gui.native_ext;
+
+import com.sun.jna.Native;
+import com.sun.jna.NativeLong;
+import com.sun.jna.Pointer;
+import com.sun.jna.WString;
+import com.sun.jna.ptr.PointerByReference;
+import net.sf.jabref.Globals;
+import net.sf.jabref.logic.util.OS;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+/**
+ * Native extensions for Windows.
+ */
+public class WindowsExtensions {
+    private static final Log LOGGER = LogFactory.getLog(WindowsExtensions.class);
+
+    // Register native calls for pin to taskbar functionality
+    static {
+        if (OS.WINDOWS) {
+            LOGGER.info("Registered Shell32 DLL");
+            Native.register("shell32");
+        }
+    }
+
+    public static void enablePinToTaskbar() {
+        // Set application user model id so that pinning JabRef to the Win7/8 taskbar works
+        // Based on http://stackoverflow.com/a/1928830
+        // Only do this if Windows is capable >= Windows 7
+        if(supportsPinToTaskbar()) {
+            setCurrentProcessExplicitAppUserModelID("JabRef." + Globals.BUILD_INFO.getVersion());
+        }
+    }
+
+    // Do not use this code in release version, it contains some memory leaks
+    private static String getCurrentProcessExplicitAppUserModelID() {
+        final PointerByReference r = new PointerByReference();
+
+        if (GetCurrentProcessExplicitAppUserModelID(r).longValue() == 0) {
+            final Pointer p = r.getValue();
+            return p.getString(0, true); // here we leak native memory by lazyness
+        }
+        return "N/A";
+    }
+
+    private static void setCurrentProcessExplicitAppUserModelID(final String appID) {
+        if (SetCurrentProcessExplicitAppUserModelID(new WString(appID)).longValue() != 0) {
+            throw new RuntimeException("unable to set current process explicit AppUserModelID to: " + appID);
+        }
+    }
+
+    private static native NativeLong GetCurrentProcessExplicitAppUserModelID(PointerByReference appID);
+
+    private static native NativeLong SetCurrentProcessExplicitAppUserModelID(WString appID);
+
+    private static boolean supportsPinToTaskbar() {
+        if (!OS.WINDOWS) {
+            return false;
+        }
+
+        try {
+            Float version = Float.parseFloat(System.getProperty("os.version"));
+            // Windows 7 == 6.1
+            return version >= 6.1;
+        } catch (NumberFormatException ex) {
+            return false;
+        }
+    }
+}

src/main/java/net/sf/jabref/logic/util/OS.java

@@ -4,7 +4,6 @@
  * Operating system (OS) detection
  */
 public class OS {
-
     // TODO: what OS do we support?
     // https://commons.apache.org/proper/commons-lang/javadocs/api-2.6/org/apache/commons/lang/SystemUtils.html
     public static final String osName = System.getProperty("os.name", "unknown").toLowerCase();