You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ba...@apache.org on 2009/05/07 10:41:43 UTC
svn commit: r772551 -
/commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
Author: bayard
Date: Thu May 7 08:41:42 2009
New Revision: 772551
URL: http://svn.apache.org/viewvc?rev=772551&view=rev
Log:
Genericizing excludeFieldList variable and updating for loop to foreach - LANG-336
Modified:
commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java?rev=772551&r1=772550&r2=772551&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java (original)
+++ commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java Thu May 7 08:41:42 2009
@@ -179,10 +179,9 @@
try {
register(object);
Field[] fields = clazz.getDeclaredFields();
- List excludedFieldList = excludeFields != null ? Arrays.asList(excludeFields) : Collections.EMPTY_LIST;
+ List<String> excludedFieldList = excludeFields != null ? Arrays.asList(excludeFields) : Collections.EMPTY_LIST;
AccessibleObject.setAccessible(fields, true);
- for (int i = 0; i < fields.length; i++) {
- Field field = fields[i];
+ for (Field field : fields) {
if (!excludedFieldList.contains(field.getName())
&& (field.getName().indexOf('$') == -1)
&& (useTransients || !Modifier.isTransient(field.getModifiers()))