From 05008212fbb49fbae7f5a320f9028ea647a2e73f Mon Sep 17 00:00:00 2001 From: Michael Koch Date: Thu, 13 Feb 2003 07:33:40 +0000 Subject: [PATCH] 2003-02-13 Michael Koch * java/net/NetPermission.java (NetPermission): Make doucmentation match the method declaration. * java/net/NetworkInterface.java (equals): Reformated for GNU coding style. * java/net/ServerSocket.java: Merged with classpath. * java/net/Socket.java: Partly merged with classpath (Added some @since). * java/net/SocketImpl.java (localPort): Merged with classpath (initialize with -1). * java/net/SocketPermission.java: Merged with classpath (reindented). * java/net/URLDecoder.java: Merged with classpath (reindented). From-SVN: r62824 --- ChangeLog | 13 +++ libjava/java/net/NetPermission.java | 2 +- libjava/java/net/NetworkInterface.java | 5 +- libjava/java/net/ServerSocket.java | 2 +- libjava/java/net/Socket.java | 41 ++++++++- libjava/java/net/SocketImpl.java | 2 +- libjava/java/net/SocketPermission.java | 112 ++++++++++++------------- libjava/java/net/URLDecoder.java | 6 +- 8 files changed, 117 insertions(+), 66 deletions(-) diff --git a/ChangeLog b/ChangeLog index f9b594821ac..884095ae68b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,16 @@ +2003-02-13 Michael Koch + + * java/net/NetPermission.java + (NetPermission): Make doucmentation match the method declaration. + * java/net/NetworkInterface.java + (equals): Reformated for GNU coding style. + * java/net/ServerSocket.java: Merged with classpath. + * java/net/Socket.java: Partly merged with classpath (Added some @since). + * java/net/SocketImpl.java + (localPort): Merged with classpath (initialize with -1). + * java/net/SocketPermission.java: Merged with classpath (reindented). + * java/net/URLDecoder.java: Merged with classpath (reindented). + 2003-02-08 Laurynas Biveinis * MAINTAINERS: Update my email. diff --git a/libjava/java/net/NetPermission.java b/libjava/java/net/NetPermission.java index 5b9502b7abe..1807d457d6c 100644 --- a/libjava/java/net/NetPermission.java +++ b/libjava/java/net/NetPermission.java @@ -64,7 +64,7 @@ public final class NetPermission extends BasicPermission /** * Initializes a new instance of NetPermission with the - * specified name and value. Note that the value field is irrelevant and is + * specified name and perms. Note that the perms field is irrelevant and is * ignored. This constructor should never need to be used. * * @param name The name of this permission diff --git a/libjava/java/net/NetworkInterface.java b/libjava/java/net/NetworkInterface.java index 60a7d764cbc..32c2cd53db0 100644 --- a/libjava/java/net/NetworkInterface.java +++ b/libjava/java/net/NetworkInterface.java @@ -200,8 +200,9 @@ public final class NetworkInterface return false; NetworkInterface tmp = (NetworkInterface) obj; - return name.equals (tmp.name) && - inetAddresses.equals (tmp.inetAddresses); + + return (name.equals (tmp.name) + && inetAddresses.equals (tmp.inetAddresses)); } /** diff --git a/libjava/java/net/ServerSocket.java b/libjava/java/net/ServerSocket.java index e2f8e637d09..0285c128c77 100644 --- a/libjava/java/net/ServerSocket.java +++ b/libjava/java/net/ServerSocket.java @@ -356,7 +356,7 @@ public class ServerSocket } catch (SocketException e) { - return false; + return false; } return true; diff --git a/libjava/java/net/Socket.java b/libjava/java/net/Socket.java index e57c1ed5381..005fd0461b2 100644 --- a/libjava/java/net/Socket.java +++ b/libjava/java/net/Socket.java @@ -95,6 +95,7 @@ public class Socket * might want this behavior. * * @specnote This constructor is public since JDK 1.4 + * @since 1.1 */ public Socket () { @@ -122,6 +123,8 @@ public class Socket * Socket * * @exception SocketException If an error occurs + * + * @since 1.1 */ protected Socket (SocketImpl impl) throws SocketException { @@ -180,6 +183,8 @@ public class Socket * exists and does not allow a connection to the specified host/port or * binding to the specified local host/port. * @exception IOException If a connection error occurs. + * + * @since 1.1 */ public Socket (String host, int port, InetAddress localAddr, int localPort) throws IOException @@ -200,6 +205,8 @@ public class Socket * @exception IOException If an error occurs * @exception SecurityException If a security manager exists and its * checkConnect method doesn't allow the operation + * + * @since 1.1 */ public Socket (InetAddress address, int port, InetAddress localAddr, int localPort) throws IOException @@ -396,6 +403,8 @@ public class Socket * is not connected, then null is returned. * * @return The local address + * + * @since 1.1 */ public InetAddress getLocalAddress () { @@ -521,6 +530,8 @@ public class Socket * @param on true to enable, false to disable * * @exception SocketException If an error occurs or Socket is not connected + * + * @since 1.1 */ public void setTcpNoDelay (boolean on) throws SocketException { @@ -539,6 +550,8 @@ public class Socket * @return Whether or not TCP_NODELAY is set * * @exception SocketException If an error occurs or Socket not connected + * + * @since 1.1 */ public boolean getTcpNoDelay() throws SocketException { @@ -567,6 +580,8 @@ public class Socket * * @exception SocketException If an error occurs or Socket not connected * @exception IllegalArgumentException If linger is negative + * + * @since 1.1 */ public void setSoLinger(boolean on, int linger) throws SocketException { @@ -602,6 +617,8 @@ public class Socket * if SO_LINGER not set * * @exception SocketException If an error occurs or Socket is not connected + * + * @since 1.1 */ public int getSoLinger() throws SocketException { @@ -681,6 +698,8 @@ public class Socket * 0 if not set * * @exception SocketException If an error occurs or Socket not connected + * + * @since 1.1 */ public synchronized void setSoTimeout (int timeout) throws SocketException { @@ -707,6 +726,8 @@ public class Socket * if not set * * @exception SocketException If an error occurs or Socket not connected + * + * @since 1.1 */ public synchronized int getSoTimeout () throws SocketException { @@ -822,7 +843,7 @@ public class Socket * * @exception SocketException If an error occurs or Socket is not connected * - * @since Java 1.3 + * @since 1.3 */ public void setKeepAlive (boolean on) throws SocketException { @@ -840,7 +861,7 @@ public class Socket * * @exception SocketException If an error occurs or Socket is not connected * - * @since Java 1.3 + * @since 1.3 */ public boolean getKeepAlive () throws SocketException { @@ -918,6 +939,8 @@ public class Socket * Closes the input side of the socket stream. * * @exception IOException If an error occurs. + * + * @since 1.3 */ public void shutdownInput() throws IOException { @@ -931,6 +954,8 @@ public class Socket * Closes the output side of the socket stream. * * @exception IOException If an error occurs. + * + * @since 1.3 */ public void shutdownOutput() throws IOException { @@ -944,6 +969,8 @@ public class Socket * Returns the socket channel associated with this socket. * * It returns null if no associated socket exists. + * + * @since 1.4 */ public SocketChannel getChannel() { @@ -1032,6 +1059,8 @@ public class Socket /** * Checks if the socket is connected + * + * @since 1.4 */ public boolean isConnected () { @@ -1040,6 +1069,8 @@ public class Socket /** * Checks if the socket is already bound. + * + * @since 1.4 */ public boolean isBound () { @@ -1048,6 +1079,8 @@ public class Socket /** * Checks if the socket is closed. + * + * @since 1.4 */ public boolean isClosed () { @@ -1056,6 +1089,8 @@ public class Socket /** * Checks if the socket's input stream is shutdown + * + * @since 1.4 */ public boolean isInputShutdown () { @@ -1064,6 +1099,8 @@ public class Socket /** * Checks if the socket's output stream is shutdown + * + * @since 1.4 */ public boolean isOutputShutdown () { diff --git a/libjava/java/net/SocketImpl.java b/libjava/java/net/SocketImpl.java index f0f1a3c87bc..12dcb0b9a5c 100644 --- a/libjava/java/net/SocketImpl.java +++ b/libjava/java/net/SocketImpl.java @@ -74,7 +74,7 @@ public abstract class SocketImpl implements SocketOptions /** * The port number the socket is bound to locally */ - protected int localport; + protected int localport = -1; /** * The port number of the remote end of the socket connection diff --git a/libjava/java/net/SocketPermission.java b/libjava/java/net/SocketPermission.java index 18fb7ef2f1f..6fa65f8793a 100644 --- a/libjava/java/net/SocketPermission.java +++ b/libjava/java/net/SocketPermission.java @@ -276,62 +276,62 @@ public final class SocketPermission extends Permission // Get ours if (hostport.indexOf(":") == -1) { - ourfirstport = 0; - ourlastport = 65535; + ourfirstport = 0; + ourlastport = 65535; } else { - // FIXME: Needs bulletproofing. - // This will dump if hostport if all sorts of bad data was passed to - // the constructor - String range = hostport.substring(hostport.indexOf(":") + 1); - if (range.startsWith("-")) - ourfirstport = 0; - else if (range.indexOf("-") == -1) - ourfirstport = Integer.parseInt(range); - else - ourfirstport = - Integer.parseInt(range.substring(0, range.indexOf("-"))); - - if (range.endsWith("-")) - ourlastport = 65535; - else if (range.indexOf("-") == -1) - ourlastport = Integer.parseInt(range); - else - ourlastport = - Integer.parseInt(range. - substring(range.indexOf("-") + 1, - range.length())); + // FIXME: Needs bulletproofing. + // This will dump if hostport if all sorts of bad data was passed to + // the constructor + String range = hostport.substring(hostport.indexOf(":") + 1); + if (range.startsWith("-")) + ourfirstport = 0; + else if (range.indexOf("-") == -1) + ourfirstport = Integer.parseInt(range); + else + ourfirstport = + Integer.parseInt(range.substring(0, range.indexOf("-"))); + + if (range.endsWith("-")) + ourlastport = 65535; + else if (range.indexOf("-") == -1) + ourlastport = Integer.parseInt(range); + else + ourlastport = + Integer.parseInt(range. + substring(range.indexOf("-") + 1, + range.length())); } // Get theirs if (p.hostport.indexOf(":") == -1) { - theirfirstport = 0; - ourlastport = 65535; + theirfirstport = 0; + ourlastport = 65535; } else { - // This will dump if hostport if all sorts of bad data was passed to - // the constructor - String range = p.hostport.substring(hostport.indexOf(":") + 1); - if (range.startsWith("-")) - theirfirstport = 0; - else if (range.indexOf("-") == -1) - theirfirstport = Integer.parseInt(range); - else - theirfirstport = - Integer.parseInt(range.substring(0, range.indexOf("-"))); - - if (range.endsWith("-")) - theirlastport = 65535; - else if (range.indexOf("-") == -1) - theirlastport = Integer.parseInt(range); - else - theirlastport = - Integer.parseInt(range. - substring(range.indexOf("-") + 1, - range.length())); + // This will dump if hostport if all sorts of bad data was passed to + // the constructor + String range = p.hostport.substring(hostport.indexOf(":") + 1); + if (range.startsWith("-")) + theirfirstport = 0; + else if (range.indexOf("-") == -1) + theirfirstport = Integer.parseInt(range); + else + theirfirstport = + Integer.parseInt(range.substring(0, range.indexOf("-"))); + + if (range.endsWith("-")) + theirlastport = 65535; + else if (range.indexOf("-") == -1) + theirlastport = Integer.parseInt(range); + else + theirlastport = + Integer.parseInt(range. + substring(range.indexOf("-") + 1, + range.length())); } // Now check them @@ -361,17 +361,17 @@ public final class SocketPermission extends Permission String ourcanonical = null, theircanonical = null; try { - ourcanonical = InetAddress.getByName(ourhost).getHostName(); - theircanonical = InetAddress.getByName(theirhost).getHostName(); + ourcanonical = InetAddress.getByName(ourhost).getHostName(); + theircanonical = InetAddress.getByName(theirhost).getHostName(); } catch (UnknownHostException e) { - // Who didn't resolve? Just assume current address is canonical enough - // Is this ok to do? - if (ourcanonical == null) - ourcanonical = ourhost; - if (theircanonical == null) - theircanonical = theirhost; + // Who didn't resolve? Just assume current address is canonical enough + // Is this ok to do? + if (ourcanonical == null) + ourcanonical = ourhost; + if (theircanonical == null) + theircanonical = theirhost; } if (ourcanonical.equals(theircanonical)) @@ -380,9 +380,9 @@ public final class SocketPermission extends Permission // Well, last chance. Try for a wildcard if (ourhost.indexOf("*.") != -1) { - String wild_domain = ourhost.substring(ourhost.indexOf("*" + 1)); - if (theircanonical.endsWith(wild_domain)) - return (true); + String wild_domain = ourhost.substring(ourhost.indexOf("*" + 1)); + if (theircanonical.endsWith(wild_domain)) + return (true); } // Didn't make it diff --git a/libjava/java/net/URLDecoder.java b/libjava/java/net/URLDecoder.java index 36747409983..0a9067160b2 100644 --- a/libjava/java/net/URLDecoder.java +++ b/libjava/java/net/URLDecoder.java @@ -82,12 +82,12 @@ public class URLDecoder { try { - return decode(s, "UTF-8"); + return decode(s, "UTF-8"); } catch (UnsupportedEncodingException uee) { - // Should never happen since UTF-8 encoding should always be supported - return s; + // Should never happen since UTF-8 encoding should always be supported + return s; } } -- 2.30.2