You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by mb...@apache.org on 2008/05/30 08:33:29 UTC
svn commit: r661579 -
/incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java
Author: mbaessler
Date: Thu May 29 23:33:28 2008
New Revision: 661579
URL: http://svn.apache.org/viewvc?rev=661579&view=rev
Log:
UIMA-518
fix test main() to not throw NPE.
https://issues.apache.org/jira/browse/UIMA-518
Modified:
incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java
Modified: incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java
URL: http://svn.apache.org/viewvc/incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java?rev=661579&r1=661578&r2=661579&view=diff
==============================================================================
--- incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java (original)
+++ incubator/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/internal/util/BrowserUtil.java Thu May 29 23:33:28 2008
@@ -98,7 +98,9 @@
String url = (args.length > 0) ? args[0] : "http://apache.org";
try {
Process process = BrowserUtil.openUrlInDefaultBrowser(url);
- process.waitFor();
+ if(process != null) {
+ process.waitFor();
+ }
} catch (Exception e) {
System.err.println("Error in BrowserUtil.main():");
e.printStackTrace(System.err);
@@ -117,7 +119,8 @@
* @param url
* The URL to open
* @return
- * Returns the process browser object or null if no browser could be found
+ * Returns the process browser object or null if no browser could be found.
+ * On MacOs null is returned in any case.
*
* @throws Exception
* If the available web browser does not run