[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

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index fb55ee6a3ccc0634d45bb7dab7a0b767517bd30f..e0655c05977e0cf406d884c7001a83bc9f0ded85 100644 (file)
--- a/libjava/ChangeLog
+++ b/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,

diff --git a/libjava/java/lang/reflect/AccessibleObject.java b/libjava/java/lang/reflect/AccessibleObject.java
index 4fb89bb44b5fa4b89421cfdd498018ebbd39546b..5b5b5f6ff522fa856716000cb1178911edc4a176 100644 (file)
--- a/libjava/java/lang/reflect/AccessibleObject.java
+++ b/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

diff --git a/libjava/java/lang/reflect/Proxy.java b/libjava/java/lang/reflect/Proxy.java
index 66373c5794fa54aeccb6ca99a6c83fb2a9d95344..8aee02fb47fd6918f2954a273d99026adce7f558 100644 (file)
--- a/libjava/java/lang/reflect/Proxy.java
+++ b/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,15 +38,16 @@ exception statement from your version. */
 
 package java.lang.reflect;
 
+import gnu.classpath.Configuration;
+import gnu.java.lang.reflect.TypeSignature;
+
 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.util.Map;
+import java.util.Set;
 
 /**
  * This class allows you to dynamically create an instance of any (or
@@ -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 unnamed (aka default) package.
+     * The package this class is in <b>including the trailing dot</b>
+     * 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.

diff --git a/libjava/java/lang/reflect/ReflectPermission.java b/libjava/java/lang/reflect/ReflectPermission.java
index 0a91ba51a4866e56f43939588bb1f6f321c74e51..400ca25afcc2ad20fcb3e18205e4a57cdabac895 100644 (file)
--- a/libjava/java/lang/reflect/ReflectPermission.java
+++ b/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