Locale.java (Locale): Don't explicitly check for null.

* java/util/Locale.java (Locale): Don't explicitly check for
	null.
	* java/util/Hashtable.java (containsKey): Don't explicitly check
	for null.
	(get): Likewise.
	* java/util/BitSet.java (and, or, xor): Don't explicitly check for
	null.
	* java/util/zip/ZipEntry.java (ZipEntry): Don't explicitly check
	for null.
	* java/text/StringCharacterIterator.java
	(StringCharacterIterator): Don't check for null.
	* java/text/ChoiceFormat.java (setChoices): Don't explicitly check
	for null pointer.
	* java/net/MulticastSocket.java (joinGroup): Don't explicitly
	check for null pointer.
	(leaveGroup): Likewise.
	* java/net/DatagramPacket.java (DatagramPacket): Removed erroneous
	comment.
	(setData): Likewise.
	* java/lang/ThreadGroup.java (ThreadGroup): Don't explicitly check
	for `p==null'.

From-SVN: r33671

libjava/java/lang/ThreadGroup.java

@@ -1,6 +1,6 @@
 // ThreadGroup.java - ThreadGroup class.
 
-/* Copyright (C) 1998, 1999  Free Software Foundation
+/* Copyright (C) 1998, 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -344,8 +344,6 @@ public class ThreadGroup
   public ThreadGroup (ThreadGroup p, String n)
   {
     checkAccess ();
-    if (p == null)
-      throw new NullPointerException ();
     if (p.destroyed_flag)
       throw new IllegalArgumentException ();
 

libjava/java/net/DatagramPacket.java

@@ -1,6 +1,6 @@
 // DatagramPacket.java - Represents packets in a connectionless protocol.
 
-/* Copyright (C) 1999  Free Software Foundation
+/* Copyright (C) 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -32,8 +32,6 @@ public final class DatagramPacket
   // JDK1.2
   public DatagramPacket(byte[] buf, int offset, int length)
   {
-    // FIXME: We can't currently rely on NullPointerException being
-    // thrown when we invoke a method on a null object.
     if (buf == null)
       throw new NullPointerException("Null buffer");
     if (offset < 0)
@@ -60,8 +58,6 @@ public final class DatagramPacket
   public DatagramPacket(byte[] buf, int offset, int length,
 	InetAddress address, int port)
   {
-    // FIXME: We can't currently rely on NullPointerException being
-    // thrown when we invoke a method on a null object.
     if (buf == null)
       throw new NullPointerException("Null buffer");
     if (offset < 0)
@@ -145,8 +141,6 @@ public final class DatagramPacket
   {
     // This form of setData must be used if offset is to be changed.
 
-    // FIXME: We can't currently rely on NullPointerException being
-    // thrown when we invoke a method on a null object.
     if (buf == null)
       throw new NullPointerException("Null buffer");
     if (offset < 0)

libjava/java/net/MulticastSocket.java

@@ -1,6 +1,6 @@
 // MulticastSocket.java
 
-/* Copyright (C) 1999  Free Software Foundation
+/* Copyright (C) 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -84,10 +84,6 @@ public class MulticastSocket extends DatagramSocket
 
   public void joinGroup(InetAddress mcastaddr) throws IOException
   {
-    // FIXME: We can't currently rely on NullPointerException being
-    // thrown when we invoke a method on a null object.
-    if (mcastaddr == null)
-      throw new NullPointerException("Null address");
     if (! mcastaddr.isMulticastAddress())
       throw new IOException("Not a Multicast address");
 
@@ -100,10 +96,6 @@ public class MulticastSocket extends DatagramSocket
 
   public void leaveGroup(InetAddress mcastaddr) throws IOException
   {
-    // FIXME: We can't currently rely on NullPointerException being
-    // thrown when we invoke a method on a null object.
-    if (mcastaddr == null)
-      throw new NullPointerException("Null address");
     if (! mcastaddr.isMulticastAddress())
       throw new IOException("Not a Multicast address");
 

libjava/java/text/ChoiceFormat.java

@@ -1,6 +1,6 @@
 // ChoiceFormat.java - Formatter for `switch'-like string substitution.
 
-/* Copyright (C) 1999  Free Software Foundation
+/* Copyright (C) 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -257,8 +257,6 @@ public class ChoiceFormat extends NumberFormat
 
   public void setChoices (double[] limits, String[] strings)
     {
-      if (limits == null || strings == null)
-	throw new NullPointerException ();
       if (limits.length != strings.length)
 	throw new IllegalArgumentException ();
       this.strings = (String[]) strings.clone();

libjava/java/text/StringCharacterIterator.java

@@ -1,6 +1,6 @@
 // StringCharacterIterator.java - Iterate over string of Unicode characters.
 
-/* Copyright (C) 1999  Free Software Foundation
+/* Copyright (C) 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -106,20 +106,14 @@ public final class StringCharacterIterator implements CharacterIterator
 
   public StringCharacterIterator (String text)
     {
-      // FIXME: remove check for null once we have compiler/runtime
-      // support for NullPointerException.
-      this (text, 0, text == null ? 0 : text.length(), 0);
+      this (text, 0, text.length(), 0);
     }
   public StringCharacterIterator (String text, int pos)
     {
-      // FIXME: remove check for null once we have compiler/runtime
-      // support for NullPointerException.
-      this (text, 0, text == null ? 0 : text.length(), pos);
+      this (text, 0, text.length(), pos);
     }
   public StringCharacterIterator (String text, int begin, int end, int pos)
     {
-      if (text == null)
-	throw new NullPointerException ();
       if (begin < 0 || begin > end || end > text.length()
 	  // In 1.1 we would also throw if `pos == end'.
 	  || pos < begin || pos > end)

libjava/java/util/BitSet.java

@@ -1,6 +1,6 @@
 // BitSet - A vector of bits.
 
-/* Copyright (C) 1998, 1999  Free Software Foundation
+/* Copyright (C) 1998, 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -24,8 +24,6 @@ public final class BitSet implements Cloneable, Serializable
 {
   public void and (BitSet bs)
     {
-      if (bs == null)
-	throw new NullPointerException ();
       int max = Math.min(bits.length, bs.bits.length);
       int i;
       for (i = 0; i < max; ++i)
@@ -110,8 +108,6 @@ public final class BitSet implements Cloneable, Serializable
 
   public void or (BitSet bs)
     {
-      if (bs == null)
-	throw new NullPointerException ();
       ensure (bs.bits.length - 1);
       int i;
       for (i = 0; i < bs.bits.length; ++i)
@@ -159,8 +155,6 @@ public final class BitSet implements Cloneable, Serializable
 
   public void xor (BitSet bs)
     {
-      if (bs == null)
-	throw new NullPointerException ();
       ensure (bs.bits.length - 1);
       int i;
       for (i = 0; i < bs.bits.length; ++i)

libjava/java/util/Hashtable.java

@@ -164,7 +164,8 @@ public class Hashtable extends Dictionary implements Cloneable, Serializable
     return newTable;
   }
 
-  public synchronized boolean contains(Object value) throws NullPointerException
+  public synchronized boolean contains(Object value)
+    throws NullPointerException
   {
     // An exception is thrown here according to the JDK 1.2 doc.
     if (value == null)
@@ -180,10 +181,6 @@ public class Hashtable extends Dictionary implements Cloneable, Serializable
 
   public synchronized boolean containsKey(Object key)
   {
-    // The Map interface mandates that we throw this.
-    if (key == null)
-      throw new NullPointerException ();
-
     for (HashtableEntry elem = bucket[Math.abs(key.hashCode()
 					       % bucket.length)];
 	 elem != null; elem = elem.nextEntry)
@@ -200,11 +197,6 @@ public class Hashtable extends Dictionary implements Cloneable, Serializable
 
   public synchronized Object get(Object key)
   {
-    // The Dictionary interface mandates that get() throw a
-    // NullPointerException if key is null.
-    if (key == null)
-      throw new NullPointerException ();
-
     for (HashtableEntry elem = bucket[Math.abs (key.hashCode()
 						% bucket.length)];
 	 elem != null; elem = elem.nextEntry)
@@ -225,8 +217,10 @@ public class Hashtable extends Dictionary implements Cloneable, Serializable
   }
 
   public synchronized Object put(Object key, Object value)
-				throws NullPointerException
+    throws NullPointerException
   {
+    // We could really just check `value == null', but checking both
+    // is a bit clearer.
     if (key == null || value == null)
       throw new NullPointerException();
 

libjava/java/util/Locale.java

@@ -1,4 +1,4 @@
-/* Copyright (C) 1998, 1999  Free Software Foundation
+/* Copyright (C) 1998, 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -65,10 +65,6 @@ public final class Locale implements java.io.Serializable, Cloneable
   public Locale (String languageCode, String countryCode,
 		 String variantCode)
   {
-    // We must explicitly check the arguments.
-    if (languageCode == null || countryCode == null
-	|| variantCode == null)
-      throw new NullPointerException ();
     language = languageCode.toLowerCase();
     country = countryCode.toUpperCase();
     variant = variantCode.toUpperCase();

libjava/java/util/zip/ZipEntry.java

@@ -1,4 +1,4 @@
-/* Copyright (C) 1999  Free Software Foundation
+/* Copyright (C) 1999, 2000  Free Software Foundation
 
    This file is part of libgcj.
 
@@ -39,8 +39,6 @@ public class ZipEntry implements ZipConstants
 
   public ZipEntry (String name)
   {
-    if (name == null)
-      throw new NullPointerException ();
     if (name.length() > 65535)
       throw new IllegalArgumentException ();
     this.name = name;