XenoAmess commented on a change in pull request #25:
URL: https://github.com/apache/commons-beanutils/pull/25#discussion_r432844590
##########
File path: src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
##########
@@ -504,7 +504,7 @@ public Object getIndexedProperty(final Object bean,
readMethod = MethodUtils.getAccessibleMethod(bean.getClass(),
readMethod);
if (readMethod != null) {
final Object[] subscript = new Object[1];
- subscript[0] = Integer.valueOf(index);
+ subscript[0] = index;
Review comment:
done.
##########
File path: src/main/java/org/apache/commons/beanutils2/LazyDynaMap.java
##########
@@ -239,7 +239,7 @@ public DynaProperty getDynaProperty(final String name) {
}
/**
- * <p>Return an array of {@code ProperyDescriptors} for the properties
+ * <p>Return an array of {@code PropertyDescriptors} for the properties
Review comment:
done.
##########
File path: src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java
##########
@@ -1589,7 +1573,7 @@ public void setIndexedProperty(final Object bean, final
String name,
writeMethod = MethodUtils.getAccessibleMethod(bean.getClass(),
writeMethod);
if (writeMethod != null) {
final Object[] subscript = new Object[2];
- subscript[0] = Integer.valueOf(index);
+ subscript[0] = index;
Review comment:
done.
##########
File path:
src/main/java/org/apache/commons/beanutils2/converters/DateTimeConverter.java
##########
@@ -240,7 +240,7 @@ protected String convertToString(final Object value) throws
Throwable {
} else if (value instanceof Calendar) {
date = ((Calendar)value).getTime();
} else if (value instanceof Long) {
- date = new Date(((Long)value).longValue());
+ date = new Date((Long) value);
Review comment:
done.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
