2006-05-10 Gary Benson <gbenson@redhat.com>

	* java/lang/ThreadGroup.java (parent): Make package-private.
	* java/lang/SecurityManager.java (checkAccess(Thread)):
	Reference ThreadGroup.parent directly to avoid extra checks.
	* java/lang/SecurityManager.java (checkAccess(ThreadGroup)):
	Likewise.

3 files changed, 11 insertions, 3 deletions

diff --git a/ChangeLog b/ChangeLog
index 882b20eac..7654a9473 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2006-05-10  Gary Benson  <gbenson@redhat.com>
+
+	* java/lang/ThreadGroup.java (parent): Make package-private.
+	* java/lang/SecurityManager.java (checkAccess(Thread)):
+	Reference ThreadGroup.parent directly to avoid extra checks.
+	* java/lang/SecurityManager.java (checkAccess(ThreadGroup)):
+	Likewise.
+
 2006-05-10  Roman Kennke <kennke@aicas.com>
 
 	Reported by Ingo Proetel (proetel@aicas.com)
diff --git a/java/lang/SecurityManager.java b/java/lang/SecurityManager.java
index 30ee1be08..999fe83dd 100644
--- a/java/lang/SecurityManager.java
+++ b/java/lang/SecurityManager.java
@@ -421,7 +421,7 @@ public class SecurityManager
   public void checkAccess(Thread thread)
   {
     if (thread.getThreadGroup() != null 
-	&& thread.getThreadGroup().getParent() == null)
+	&& thread.getThreadGroup().parent == null)
       checkPermission(new RuntimePermission("modifyThread"));
   }
 
@@ -454,7 +454,7 @@ public class SecurityManager
    */
   public void checkAccess(ThreadGroup g)
   {
-    if (g.getParent() == null)
+    if (g.parent == null)
       checkPermission(new RuntimePermission("modifyThreadGroup"));
   }
 
diff --git a/java/lang/ThreadGroup.java b/java/lang/ThreadGroup.java
index 7fbef88f4..daddf43e3 100644
--- a/java/lang/ThreadGroup.java
+++ b/java/lang/ThreadGroup.java
@@ -66,7 +66,7 @@ public class ThreadGroup implements UncaughtExceptionHandler
   static boolean had_uncaught_exception;
 
   /** The parent thread group. */
-  private final ThreadGroup parent;
+  final ThreadGroup parent;
 
   /** The group name, non-null. */
   final String name;