ggregory    2003/07/18 20:25:39

  Modified:    lang/src/java/org/apache/commons/lang/builder
                        CompareToBuilder.java
  Log:
  Severity      Description     Resource        In Folder       Location        
Creation Time
        The static method setAccessible(AccessibleObject[], boolean) from the type 
AccessibleObject should be accessed directly         CompareToBuilder.java   Apache 
Jakarta Commons/lang/src/java/org/apache/commons/lang/builder    line 254        July 
18, 2003 7:11:22 PM
  
  Revision  Changes    Path
  1.17      +7 -6      
jakarta-commons/lang/src/java/org/apache/commons/lang/builder/CompareToBuilder.java
  
  Index: CompareToBuilder.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/lang/src/java/org/apache/commons/lang/builder/CompareToBuilder.java,v
  retrieving revision 1.16
  retrieving revision 1.17
  diff -u -r1.16 -r1.17
  --- CompareToBuilder.java     14 Jul 2003 22:25:03 -0000      1.16
  +++ CompareToBuilder.java     19 Jul 2003 03:25:38 -0000      1.17
  @@ -53,6 +53,7 @@
    */
   package org.apache.commons.lang.builder;
   
  +import java.lang.reflect.AccessibleObject;
   import java.lang.reflect.Field;
   import java.lang.reflect.Modifier;
   import java.util.Comparator;
  @@ -91,7 +92,7 @@
    *
    * <p>Alternatively, there is a method that uses reflection to determine
    * the fields to test. Because these fields are usually private, the method,
  - * <code>reflectionCompare</code>, uses <code>Field.setAccessible</code> to change
  + * <code>reflectionCompare</code>, uses <code>AccessibleObject.setAccessible</code> 
to change
    * the visibility of the fields. This will fail under a security manager,
    * unless the appropriate permissions are set. It is also slower than testing
    * explicitly.</p>
  @@ -134,7 +135,7 @@
        * <p>This method uses reflection to determine the ordering between two
        * Objects.</p>
        *
  -     * <p>It uses <code>Field.setAccessible</code> to gain access to private
  +     * <p>It uses <code>AccessibleObject.setAccessible</code> to gain access to 
private
        * fields. This means that it will throw a security exception if run under
        * a security manger, if the permissions are not set up correctly. It is
        * also not as efficient as testing explicitly.</p>
  @@ -161,7 +162,7 @@
        * <p>This method uses reflection to determine if the two Objects are
        * equal.</p>
        *
  -     * <p>It uses <code>Field.setAccessible</code> to gain access to private
  +     * <p>It uses <code>AccessibleObject.setAccessible</code> to gain access to 
private
        * fields. This means that it will throw a security exception if run under
        * a security manger, if  the permissions are not set up correctly. It is
        * also not as efficient as testing explicitly.</p>
  @@ -190,7 +191,7 @@
        * <p>This method uses reflection to determine if the two Objects are
        * equal.</p>
        *
  -     * <p>It uses <code>Field.setAccessible</code> to gain access to private
  +     * <p>It uses <code>AccessibleObject.setAccessible</code> to gain access to 
private
        * fields. This means that it will throw a security exception if run under
        * a security manger, if  the permissions are not set up correctly. It is
        * also not as efficient as testing explicitly.</p>
  @@ -251,7 +252,7 @@
           CompareToBuilder builder,
           boolean useTransients) {
           Field[] fields = clazz.getDeclaredFields();
  -        Field.setAccessible(fields, true);
  +        AccessibleObject.setAccessible(fields, true);
           for (int i = 0; i < fields.length && builder.comparison == 0; i++) {
               Field f = fields[i];
               if ((f.getName().indexOf('$') == -1)
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to