Changeset 6616ccd3

Timestamp:

May 25, 2011 9:05:20 PM (10 years ago)

Author:

zzz <zzz@…>

Branches:

master

Children:

f5f4f14b

Parents:

075f89f

Message:

cleanup in prep for more changes

File:

• core/java/src/net/i2p/util/NativeBigInteger.java (modified) (5 diffs)

core/java/src/net/i2p/util/NativeBigInteger.java

@@ -118 +118 @@
     private static final boolean _isOS2 = System.getProperty("os.name").startsWith("OS/2");
     private static final boolean _isMac = System.getProperty("os.name").startsWith("Mac");
-    private static final boolean _isLinux = System.getProperty("os.name").toLowerCase().indexOf("linux") != -1;
-    private static final boolean _isFreebsd = System.getProperty("os.name").toLowerCase().indexOf("freebsd") != -1;
-    private static final boolean _isNix = !(_isWin || _isMac || _isOS2);
+    private static final boolean _isLinux = System.getProperty("os.name").toLowerCase().contains("linux");
+    private static final boolean _isFreebsd = System.getProperty("os.name").toLowerCase().contains("freebsd");
+
     /* libjbigi.so vs jbigi.dll */
     private static final String _libPrefix = (_isWin || _isOS2 ? "" : "lib");
@@ -139 +139 @@
       */
     private static String resolveCPUType() {
-        boolean is64 = (-1 != System.getProperty("os.arch").indexOf("64"));
+        /*
+         * This isn't always correct.
+         * http://stackoverflow.com/questions/807263/how-do-i-detect-which-kind-of-jre-is-installed-32bit-vs-64bit
+         * http://mark.koli.ch/2009/10/javas-osarch-system-property-is-the-bitness-of-the-jre-not-the-operating-system.html
+         * http://mark.koli.ch/2009/10/reliably-checking-os-bitness-32-or-64-bit-on-windows-with-a-tiny-c-app.html
+         * sun.arch.data.model not on all JVMs
+         * sun.arch.data.model == 64 => 64 bit processor
+         * sun.arch.data.model == 32 => A 32 bit JVM but could be either 32 or 64 bit processor or libs
+         * os.arch contains "64" could be 32 or 64 bit libs
+         */
+        boolean is64 = "64".equals(System.getProperty("sun.arch.data.model")) ||
+                       System.getProperty("os.arch").contains("64");
         if (is64)
             return JBIGI_OPTIMIZATION_ATHLON64;
@@ -483 +494 @@
             outFile = new File(I2PAppContext.getGlobalContext().getTempDir(), filename);
             fos = new FileOutputStream(outFile);
-            // wtf this was 4096*1024 which is really excessive for a roughly 50KB file
             byte buf[] = new byte[4096];
             while (true) {
@@ -517 +527 @@
     }
     
+    /**
+     *  @return may be null if optimized is true
+     */
     private static final String getResourceName(boolean optimized) {
-        String pref = _libPrefix;
         String middle = getMiddleName(optimized);
-        String suff = _libSuffix;
-        if(pref == null || middle == null || suff == null)
+        if (middle == null)
             return null;
-        return pref+middle+suff;
-    }
-    
-    private static final String getMiddleName(boolean optimized){
-        
+        return _libPrefix + middle + _libSuffix;
+    }
+    
+    /**
+     *  @return may be null if optimized is true; returns jbigi-xxx-none if optimize is false
+     */
+    private static final String getMiddleName(boolean optimized) {
         String sAppend;
-        if(optimized)
-        {
-            if(sCPUType == null)
+        if (optimized) {
+            if (sCPUType == null)
                 return null;
-            else
-                sAppend = "-"+sCPUType;        
-        }else
-               sAppend = "-none";
+            sAppend = "-" + sCPUType;        
+        } else {
+            sAppend = "-none";
+        }
 
         if(_isWin)
              return "jbigi-windows"+sAppend; // The convention on Windows
-        if(_isLinux)
-            return "jbigi-linux"+sAppend; // The convention on linux...
         if(_isFreebsd)
             return "jbigi-freebsd"+sAppend; // The convention on freebsd...
@@ -548 +558 @@
         if(_isOS2)
             return "jbigi-os2"+sAppend;
-        throw new RuntimeException("Dont know jbigi library name for os type '"+System.getProperty("os.name")+"'");
+        //throw new RuntimeException("Dont know jbigi library name for os type '"+System.getProperty("os.name")+"'");
+        // use linux as the default, don't throw exception
+        return "jbigi-linux" + sAppend;
     }
 }