Apache
/
commons-lang
mirror of https://github.com/apache/commons-lang.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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


git-svn-id: https://svn.apache.org/repos/asf/jakarta/commons/proper/lang/trunk@137451 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Gary D. Gregory 2003-07-19 03:25:38 +00:00
parent 4e070edf4b
commit 4bf65d4732
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/java/org/apache/commons/lang/builder/CompareToBuilder.java
View File

@ -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>
@ -107,7 +108,7 @@
* @author Stephen Colebourne
* @author Gary Gregory
* @since 1.0
* @version $Id: CompareToBuilder.java,v 1.16 2003/07/14 22:25:03 bayard Exp $
* @version $Id: CompareToBuilder.java,v 1.17 2003/07/19 03:25:38 ggregory Exp $
*/
public class CompareToBuilder {
@ -134,7 +135,7 @@ public CompareToBuilder() {
* <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 @@ public static int reflectionCompare(Object lhs, Object rhs) {
* <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 @@ public static int reflectionCompare(Object lhs, Object rhs, boolean testTransien
* <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 @@ private static void reflectionAppend(
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)