This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-lang.git
commit b7698a627faec21a4009d8d67e6b47a3d79a9cd8
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Thu Sep 12 10:17:16 2024 -0400
Remove redundant syntax
---
.../lang3/builder/ReflectionToStringBuilder.java | 42 +++++++++++-----------
1 file changed, 21 insertions(+), 21 deletions(-)
diff --git
a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
index fa95b4e7e..b0c382025 100644
---
a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
+++
b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
@@ -551,9 +551,9 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
final T object, final ToStringStyle style, final StringBuffer
buffer,
final Class<? super T> reflectUpToClass, final boolean
outputTransients, final boolean outputStatics) {
super(object, style, buffer);
- this.setUpToClass(reflectUpToClass);
- this.setAppendTransients(outputTransients);
- this.setAppendStatics(outputStatics);
+ setUpToClass(reflectUpToClass);
+ setAppendTransients(outputTransients);
+ setAppendStatics(outputStatics);
}
/**
@@ -582,10 +582,10 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
final Class<? super T> reflectUpToClass, final boolean
outputTransients, final boolean outputStatics,
final boolean excludeNullValues) {
super(object, style, buffer);
- this.setUpToClass(reflectUpToClass);
- this.setAppendTransients(outputTransients);
- this.setAppendStatics(outputStatics);
- this.setExcludeNullValues(excludeNullValues);
+ setUpToClass(reflectUpToClass);
+ setAppendTransients(outputTransients);
+ setAppendStatics(outputStatics);
+ setExcludeNullValues(excludeNullValues);
}
/**
@@ -605,11 +605,11 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
// Reject field from inner class.
return false;
}
- if (Modifier.isTransient(field.getModifiers()) &&
!this.isAppendTransients()) {
+ if (Modifier.isTransient(field.getModifiers()) &&
!isAppendTransients()) {
// Reject transient fields.
return false;
}
- if (Modifier.isStatic(field.getModifiers()) &&
!this.isAppendStatics()) {
+ if (Modifier.isStatic(field.getModifiers()) && !isAppendStatics()) {
// Reject static fields.
return false;
}
@@ -641,7 +641,7 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
*/
protected void appendFieldsIn(final Class<?> clazz) {
if (clazz.isArray()) {
- this.reflectionAppendArray(this.getObject());
+ reflectionAppendArray(getObject());
return;
}
// The elements in the returned array are not sorted and are not in
any particular order.
@@ -649,11 +649,11 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
AccessibleObject.setAccessible(fields, true);
for (final Field field : fields) {
final String fieldName = field.getName();
- if (this.accept(field)) {
+ if (accept(field)) {
try {
// Warning: Field.get(Object) creates wrappers objects
// for primitive types.
- final Object fieldValue = this.getValue(field);
+ final Object fieldValue = getValue(field);
if (!excludeNullValues || fieldValue != null) {
this.append(fieldName, fieldValue,
!field.isAnnotationPresent(ToStringSummary.class));
}
@@ -709,7 +709,7 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
* @see java.lang.reflect.Field#get(Object)
*/
protected Object getValue(final Field field) throws IllegalAccessException
{
- return field.get(this.getObject());
+ return field.get(getObject());
}
/**
@@ -749,7 +749,7 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
* @return {@code this} instance.
*/
public ReflectionToStringBuilder reflectionAppendArray(final Object array)
{
- this.getStyle().reflectionAppendArrayDetail(this.getStringBuffer(),
null, array);
+ getStyle().reflectionAppendArrayDetail(getStringBuffer(), null, array);
return this;
}
@@ -843,17 +843,17 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
*/
@Override
public String toString() {
- if (this.getObject() == null) {
- return this.getStyle().getNullText();
+ if (getObject() == null) {
+ return getStyle().getNullText();
}
validate();
- Class<?> clazz = this.getObject().getClass();
- this.appendFieldsIn(clazz);
- while (clazz.getSuperclass() != null && clazz != this.getUpToClass()) {
+ Class<?> clazz = getObject().getClass();
+ appendFieldsIn(clazz);
+ while (clazz.getSuperclass() != null && clazz != getUpToClass()) {
clazz = clazz.getSuperclass();
- this.appendFieldsIn(clazz);
+ appendFieldsIn(clazz);
}
return super.toString();
}
@@ -863,7 +863,7 @@ public class ReflectionToStringBuilder extends
ToStringBuilder {
*/
private void validate() {
if (ArrayUtils.containsAny(this.excludeFieldNames, (Object[])
this.includeFieldNames)) {
- ToStringStyle.unregister(this.getObject());
+ ToStringStyle.unregister(getObject());
throw new IllegalStateException("includeFieldNames and
excludeFieldNames must not intersect");
}
}
