re PR libgcj/4383 (file: protocol not supported in URLConnection?)

	PR libgcj/4383

	* gnu/gcj/protocol/file/Connection.java (connect): Throw
	FileNotFoundException if appropriate.
	* gnu/gcj/protocol/file/Handler.java (openConnection): Throw an
	IOException if we got a file: url with a hostname. Comment out protocol
	switch to ftp for now.
	* java/net/URL.java (URL): Include protocol name in exception
	message
	when handler can't be found.

From-SVN: r45898

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index a76e941..11791d2 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -2,6 +2,14 @@
 
        * java/lang/natClass.cc (_Jv_IsAssignableFrom): Handle the case of
        an uninitialized target class.
+       
+       * gnu/gcj/protocol/file/Connection.java (connect): Throw 
+       FileNotFoundException if appropriate.
+       * gnu/gcj/protocol/file/Handler.java (openConnection): Throw an 
+       IOException if we got a file: url with a hostname. Comment out protocol
+       switch to ftp for now.
+       * java/net/URL.java (URL): Include protocol name in exception message
+       when handler can't be found.
 
 2001-09-28  Per Bothner  <per@bothner.com>
 

diff --git a/libjava/gnu/gcj/protocol/file/Connection.java b/libjava/gnu/gcj/protocol/file/Connection.java
index b7804cb..e9ef08f 100644 (file)
--- a/libjava/gnu/gcj/protocol/file/Connection.java
+++ b/libjava/gnu/gcj/protocol/file/Connection.java
@@ -48,7 +48,11 @@ class Connection extends URLConnection
 
     // If not connected, then file needs to be openned.
     fileIn = new File(url.getFile());
-    connected = true;
+    
+    if (fileIn.exists())
+      connected = true;
+    else
+      throw new FileNotFoundException("No such file or directory");
   }
 
   public InputStream getInputStream() throws IOException

diff --git a/libjava/gnu/gcj/protocol/file/Handler.java b/libjava/gnu/gcj/protocol/file/Handler.java
index b4d8fbf..f48ef8e 100644 (file)
--- a/libjava/gnu/gcj/protocol/file/Handler.java
+++ b/libjava/gnu/gcj/protocol/file/Handler.java
@@ -35,6 +35,8 @@ public class Handler extends URLStreamHandler
     String host = url.getHost();
     if ((host != null) && (! host.equals("")))
       {
+        throw new IOException("ftp protocol handler not yet implemented.");
+        /*
        // Reset the protocol (and implicitly the handler) for this URL.
        // Then have the URL attempt the connection again, as it will
        // get the changed handler the next time around.
@@ -43,6 +45,7 @@ public class Handler extends URLStreamHandler
        // Until the ftp protocol handler is written, this will cause
        // a NullPointerException.
        return url.openConnection();
+       */
       }
 
     return new Connection(url);

diff --git a/libjava/java/net/URL.java b/libjava/java/net/URL.java
index 25291fa..636e6ef 100644 (file)
--- a/libjava/java/net/URL.java
+++ b/libjava/java/net/URL.java
@@ -76,7 +76,7 @@ public final class URL implements Serializable
       this.handler = setURLStreamHandler(protocol);
 
     if (this.handler == null)
-      throw new MalformedURLException("Handler for protocol not found");
+      throw new MalformedURLException("Protocol handler not found: " + protocol);
 
     this.host = host;
 
@@ -175,7 +175,7 @@ public final class URL implements Serializable
       this.handler = setURLStreamHandler(protocol);
 
     if (this.handler == null)
-      throw new MalformedURLException("Handler for protocol not found");
+      throw new MalformedURLException("Protocol handler not found: " + protocol);
 
     // JDK 1.2 doc for parseURL specifically states that any '#' ref
     // is to be excluded by passing the 'limit' as the indexOf the '#'