Changeset 31582 for main/trunk/gli/src/org/greenstone/gatherer/util/Utility.java

Timestamp:

2017-04-06T19:15:55+12:00 (7 years ago)

Author:

ak19

Message:

In place of the Input- and OutputStreamGobbler classes, am now shifting GLI code to use SafeProcess too, copied across from GS3 src code. Class SafeProcess includes the two streamgobblers as static inner classes and some more functionality with safely running an external process from Java.

File:

• main/trunk/gli/src/org/greenstone/gatherer/util/Utility.java (modified) (5 diffs)

main/trunk/gli/src/org/greenstone/gatherer/util/Utility.java

@@ -41 +41 @@
 // Don't even think about adding import java.awt.* here!
 // The functions in this class should not use any graphical classes. Put your function somewhere else buster!
-import java.io.Closeable;
 import java.io.*;
 import java.net.*;
@@ -50 +49 @@
 // Don't even think about adding import org.greenstone.gatherer.Gatherer in here!
 // The functions in this class should be independent of the Gatherer class. Put your function somewhere else buster!
-
+import org.greenstone.gatherer.util.SafeProcess; // for the closeResource() static method
 
 /** To provide a library of common methods, in a static context, for use in the Gatherer.
@@ -74 +73 @@
     static final public String PERL_EXECUTABLE_WINDOWS = "Perl.exe";
 
+    /** Platform independent NEWLINE character */
+    public static final String NEWLINE;    
+    
+    // NEWLINE related code copied across from GS3 src code
+    // Before Java 7, no System.lineSeparator() or System.getProperty("line.separator")
+    // And on local linux, am compiling with JDK 6, so need this.
+    // http://stackoverflow.com/questions/207947/how-do-i-get-a-platform-dependent-new-line-character
+    // http://stackoverflow.com/questions/2591083/getting-java-version-at-runtime
+    // https://www.tutorialspoint.com/java/lang/package_getspecificationversion.htm
+    // https://docs.oracle.com/javase/tutorial/essential/environment/sysprop.html
+    // Can initialise static final vars on declaration or in static initialisation code block
+    // http://stackoverflow.com/questions/2339932/java-can-final-variables-be-initialized-in-static-initialization-block
+    // Initialise object member final vars on declaration or in constructors
+    static {    
+    double java_version = Double.parseDouble(System.getProperty("java.specification.version"));
+    if(java_version >= 1.7) {
+        NEWLINE = System.getProperty("line.separator");
+    } else {
+        NEWLINE = isWindows() ? "\r\n" : "\n";
+    }   
+    }
 
     /** 
@@ -93 +113 @@
         System.err.println("*** Exception occurred: " + e.getMessage());
     } finally {
-        closeResource(fin);     
+        SafeProcess.closeResource(fin);     
     }
 
@@ -575 +595 @@
         System.err.println("Exception occurred: " + e.getMessage());
     } finally {
-        closeResource(fin);
-        closeResource(fout);        
+        SafeProcess.closeResource(fin);
+        SafeProcess.closeResource(fout);        
         
     }
     return oldValue;
     }
-
-    // For safely closing streams/handles/resources. 
-    // For examples of use look in the Input- and OutputStreamGobbler classes.
-    // http://docs.oracle.com/javase/tutorial/essential/exceptions/finally.html
-    // http://stackoverflow.com/questions/481446/throws-exception-in-finally-blocks
-    public static void closeResource(Closeable resourceHandle) {
-    try {
-        if(resourceHandle != null) {
-        resourceHandle.close();
-        resourceHandle = null;
-        }
-    } catch(Exception e) {
-        System.err.println("Exception closing resource: " + e.getMessage());
-        e.printStackTrace();
-        resourceHandle = null;
-    }
-    }
-
-    public static void closeProcess(Process prcs) {
-    if( prcs != null ) { 
-        closeResource(prcs.getErrorStream()); 
-        closeResource(prcs.getOutputStream()); 
-        closeResource(prcs.getInputStream()); 
-        prcs.destroy(); 
-    } 
-    }
 }