From: Bryce McKinlay <bryce@mckinlay.net.nz>
Date: Thu, 7 Aug 2003 01:12:27 +0000 (+0000)
Subject: Thread.java (Thread): Check for null "name" from start of private constructor...
X-Git-Tag: releases/gcc-3.4.0~4360
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=6eac0ef54eba75e34876c600199e30d290747b13;p=thirdparty%2Fgcc.git

Thread.java (Thread): Check for null "name" from start of private constructor...

	* java/lang/Thread.java (Thread): Check for null "name" from
	start of private constructor, not after calling the private
	constructor.

From-SVN: r70216
---

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index f3ab9636bda1..04307791bdee 100644
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,9 @@
+2003-08-07  Bryce McKinlay  <bryce@mckinlay.net.nz>
+
+	* java/lang/Thread.java (Thread): Check for null "name" from
+	start of private constructor, not after calling the private
+	constructor.
+
 2003-08-06  Tom Tromey  <tromey@redhat.com>
 
 	* java/io/FilePermission.java (equals): Use correct index for
diff --git a/libjava/java/lang/Thread.java b/libjava/java/lang/Thread.java
index 32f7d174580f..64498b23ba43 100644
--- a/libjava/java/lang/Thread.java
+++ b/libjava/java/lang/Thread.java
@@ -614,11 +614,6 @@ public class Thread implements Runnable
   public Thread (ThreadGroup g, Runnable r, String n)
   {
     this (currentThread (), g, r, n);
-
-    // The Class Libraries book says ``threadName cannot be null''.  I
-    // take this to mean NullPointerException.
-    if (n == null)
-      throw new NullPointerException ();
   }
 
   /**
@@ -645,15 +640,15 @@ public class Thread implements Runnable
   {
     // Just ignore stackSize for now.
     this (currentThread (), g, r, n);
+  }
 
+  private Thread (Thread current, ThreadGroup g, Runnable r, String n)
+  {
     // The Class Libraries book says ``threadName cannot be null''.  I
     // take this to mean NullPointerException.
     if (n == null)
       throw new NullPointerException ();
-  }
-
-  private Thread (Thread current, ThreadGroup g, Runnable r, String n)
-  {
+      
     if (g == null)
       {
 	// If CURRENT is null, then we are bootstrapping the first thread.