[multiple changes]

2004-10-18  Michael Koch  <konqueror@gmx.de>

	* java/lang/reflect/Proxy.java: Improved javadocs.

2004-10-18  Michael Koch  <konqueror@gmx.de>

	* java/lang/reflect/AccessibleObject.java
	(checkPermission): Removed redundant final modifier.
	(secureSetAccessible): Likewise.
	* java/lang/reflect/Proxy.java:
	Reworked import statements.
	(generate): Removed redundant final modifier.
	* java/lang/reflect/ReflectPermission.java:
	Reorder package declaration and import statement.

2004-10-18  Jeroen Frijters  <jeroen@frijters.net>

	* java/lang/reflect/Proxy.java
	(count): Removed useless initializer.

From-SVN: r89201

libjava/ChangeLog

@@ -1,3 +1,23 @@
+2004-10-18  Michael Koch  <konqueror@gmx.de>
+
+	* java/lang/reflect/Proxy.java: Improved javadocs.
+
+2004-10-18  Michael Koch  <konqueror@gmx.de>
+
+	* java/lang/reflect/AccessibleObject.java
+	(checkPermission): Removed redundant final modifier.
+	(secureSetAccessible): Likewise.
+	* java/lang/reflect/Proxy.java:
+	Reworked import statements.
+	(generate): Removed redundant final modifier.
+	* java/lang/reflect/ReflectPermission.java:
+	Reorder package declaration and import statement.
+
+2004-10-18  Jeroen Frijters  <jeroen@frijters.net>
+
+	* java/lang/reflect/Proxy.java
+	(count): Removed useless initializer.
+
 2004-10-17  Michael Koch  <konqueror@gmx.de>
 
 	* java/net/BindException.java,

libjava/java/lang/reflect/AccessibleObject.java

@@ -133,7 +133,7 @@ public class AccessibleObject
    *
    * @throws SecurityException if permission is denied
    */
-  private static final void checkPermission()
+  private static void checkPermission()
   {
     SecurityManager sm = System.getSecurityManager();
     if (sm != null)
@@ -148,7 +148,7 @@ public class AccessibleObject
    * @throws SecurityException if flag is true and this is a constructor
    * for <code>java.lang.Class</code>.
    */
-  private final void secureSetAccessible(boolean flag)
+  private void secureSetAccessible(boolean flag)
   {
     if (flag &&
         (this instanceof Constructor

libjava/java/lang/reflect/Proxy.java

@@ -1,5 +1,5 @@
 /* Proxy.java -- build a proxy class that implements reflected interfaces
-   Copyright (C) 2001, 2002, 2003 Free Software Foundation, Inc.
+   Copyright (C) 2001, 2002, 2003, 2004  Free Software Foundation, Inc.
 
 This file is part of GNU Classpath.
 
@@ -38,16 +38,17 @@ exception statement from your version. */
 
 package java.lang.reflect;
 
-import java.io.Serializable;
-import java.security.ProtectionDomain;
-import java.util.Map;
-import java.util.HashMap;
-import java.util.Set;
-import java.util.HashSet;
-import java.util.Iterator;
 import gnu.classpath.Configuration;
 import gnu.java.lang.reflect.TypeSignature;
 
+import java.io.Serializable;
+import java.security.ProtectionDomain;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Set;
+
 /**
  * This class allows you to dynamically create an instance of any (or
  * even multiple) interfaces by reflection, and decide at runtime
@@ -153,7 +154,7 @@ import gnu.java.lang.reflect.TypeSignature;
  * @see InvocationHandler
  * @see UndeclaredThrowableException
  * @see Class
- * @author Eric Blake <ebb9@email.byu.edu>
+ * @author Eric Blake (ebb9@email.byu.edu)
  * @since 1.3
  * @status updated to 1.4, except for the use of ProtectionDomain
  */
@@ -722,8 +723,8 @@ public class Proxy implements Serializable
   private static final class ProxyData
   {
     /**
-     * The package this class is in *including the trailing dot* or "" for
+     * The package this class is in <b>including the trailing dot</b>
-     * the unnamed (aka default) package.
+     * or an empty string for the unnamed (aka default) package.
      */
     String pack;
 
@@ -754,7 +755,7 @@ public class Proxy implements Serializable
     /**
      * For unique id's
      */
-    private static int count = 0;
+    private static int count;
 
     /**
      * The id of this proxy class
@@ -771,7 +772,7 @@ public class Proxy implements Serializable
     /**
      * Return the name of a package (including the trailing dot)
      * given the name of a class.
-     * Returns "" if no package.  We use this in preference to
+     * Returns an empty string if no package.  We use this in preference to
      * using Class.getPackage() to avoid problems with ClassLoaders
      * that don't set the package.
      */
@@ -1302,7 +1303,7 @@ public class Proxy implements Serializable
      *        implies the bootstrap class loader
      * @return the proxy class Class object
      */
-    final Class generate(ClassLoader loader)
+    Class generate(ClassLoader loader)
     {
       byte[] bytecode = new byte[pool.length() + stream.length()];
       // More efficient to bypass calling charAt() repetitively.

libjava/java/lang/reflect/ReflectPermission.java

@@ -35,15 +35,15 @@ this exception to your version of the library, but you are not
 obligated to do so.  If you do not wish to do so, delete this
 exception statement from your version. */
 
+package java.lang.reflect;
+
+import java.security.BasicPermission;
+
 /* Written using "Java Class Libraries", 2nd edition, ISBN 0-201-31002-3
  * "The Java Language Specification", ISBN 0-201-63451-1
  * plus online API docs for JDK 1.2 beta from http://www.javasoft.com.
  */
 
-package java.lang.reflect;
-
-import java.security.BasicPermission;
-
 /**
  * This class implements permissions for reflection.  This is a named
  * permission, and the only defined name is suppressAccessChecks, which