[multiple changes]

2004-07-17  Jeroen Frijters  <jeroen@frijters.net>

	* java/net/DatagramPacket.java (setAddress): Removed check for
	null address.

2004-07-17  Michael Koch  <konqueror@gmx.de>

	* java/net/DatagramSocket.java
	(getLocalAddress): Check if socket is bound or not.
	* java/net/Socket.java
	(getLocalAddrss): Check if socket is bound or not.
	(getPort): Return -1 when not connected. Dont check getImpl() for
	null.
	(setReuseAddress): Check if socket is closed.
	(isConnected): Check if getImpl() returns null.

From-SVN: r84864

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index cbb023adcc672478137f076ea07933b76a3590d9..b73fa4609785b5fe27ab245bc3db6a02fde35e14 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,19 @@
+2004-07-17  Jeroen Frijters  <jeroen@frijters.net>
+
+       * java/net/DatagramPacket.java (setAddress): Removed check for
+       null address.
+
+2004-07-17  Michael Koch  <konqueror@gmx.de>
+
+       * java/net/DatagramSocket.java
+       (getLocalAddress): Check if socket is bound or not.
+       * java/net/Socket.java
+       (getLocalAddrss): Check if socket is bound or not.
+       (getPort): Return -1 when not connected. Dont check getImpl() for
+       null.
+       (setReuseAddress): Check if socket is closed.
+       (isConnected): Check if getImpl() returns null.
+
 2004-07-17  Mark Wielaard  <mark@klomp.org>
 
        * java/awt/event/InvocationEvent.java (dispatch): Synchronize

diff --git a/libjava/java/net/DatagramPacket.java b/libjava/java/net/DatagramPacket.java
index 8ff64435c2bb9f9edb0abe3bb229f2b484dc097c..549a3609a3dcf44b9abe4eb0c37673a79f4659aa 100644 (file)
--- a/libjava/java/net/DatagramPacket.java
+++ b/libjava/java/net/DatagramPacket.java
@@ -278,9 +278,6 @@ public final class DatagramPacket
    */
   public synchronized void setAddress(InetAddress address)
   {
-    if (address == null)
-      throw new NullPointerException("Null address");
-
     this.address = address;
   }
 

diff --git a/libjava/java/net/DatagramSocket.java b/libjava/java/net/DatagramSocket.java
index 2808d82428b5876792cb58f3e8f4cb68f03b24d0..d29d122349838aef8d14d5ee54e7b446e0b60a06 100644 (file)
--- a/libjava/java/net/DatagramSocket.java
+++ b/libjava/java/net/DatagramSocket.java
@@ -1,5 +1,6 @@
 /* DatagramSocket.java -- A class to model UDP sockets
-   Copyright (C) 1998, 1999, 2000, 2002, 2003 Free Software Foundation, Inc.
+   Copyright (C) 1998, 1999, 2000, 2002, 2003, 2004
+   Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -282,7 +283,7 @@ public class DatagramSocket
    */
   public InetAddress getLocalAddress()
   {
-    if (isClosed())
+    if (! isBound())
       return null;
 
     InetAddress localAddr;

diff --git a/libjava/java/net/Socket.java b/libjava/java/net/Socket.java
index 43c6f3c3d50cf2d1d9765886e889f7f79ed58d11..714a958e6aaaa27d418dcda2903f6603816ccf94 100644 (file)
--- a/libjava/java/net/Socket.java
+++ b/libjava/java/net/Socket.java
@@ -488,6 +488,9 @@ public class Socket
    */
   public InetAddress getLocalAddress()
   {
+    if (! isBound())
+      return null;
+
     InetAddress addr = null;
 
     try
@@ -523,12 +526,11 @@ public class Socket
   public int getPort()
   {
     if (! isConnected())
-      return 0;
+      return -1;
 
     try
       {
-       if (getImpl() != null)
-         return getImpl().getPort();
+       return getImpl().getPort();
       }
     catch (SocketException e)
       {
@@ -1155,6 +1157,9 @@ public class Socket
    */
   public void setReuseAddress(boolean reuseAddress) throws SocketException
   {
+    if (isClosed())
+      throw new SocketException("socket is closed");
+
     getImpl().setOption(SocketOptions.SO_REUSEADDR,
                         Boolean.valueOf(reuseAddress));
   }
@@ -1217,6 +1222,9 @@ public class Socket
   {
     try
       {
+       if (getImpl() == null)
+         return false;
+
        return getImpl().getInetAddress() != null;
       }
     catch (SocketException e)