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-beanutils.git
The following commit(s) were added to refs/heads/master by this push: new 5367200f Javadoc and comments 5367200f is described below commit 5367200fbce8690c75bb58d0b04c77c86c367019 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Mon Jan 15 10:16:05 2024 -0500 Javadoc and comments --- .../org/apache/commons/beanutils2/ContextClassLoaderLocal.java | 2 +- .../java/org/apache/commons/beanutils2/PropertyUtilsBean.java | 4 ++-- .../org/apache/commons/beanutils2/locale/BaseLocaleConverter.java | 2 +- .../java/org/apache/commons/beanutils2/locale/package-info.java | 2 +- .../java/org/apache/commons/beanutils2/BeanificationTestCase.java | 6 +++--- .../java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java | 4 ++-- .../commons/beanutils2/locale/LocaleBeanificationTestCase.java | 8 ++++---- 7 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/main/java/org/apache/commons/beanutils2/ContextClassLoaderLocal.java b/src/main/java/org/apache/commons/beanutils2/ContextClassLoaderLocal.java index d9d122af..affb4e34 100644 --- a/src/main/java/org/apache/commons/beanutils2/ContextClassLoaderLocal.java +++ b/src/main/java/org/apache/commons/beanutils2/ContextClassLoaderLocal.java @@ -88,7 +88,7 @@ import java.util.WeakHashMap; * related classes and static variables cannot be garbage-collected. This is * not expected to be an issue with the commons-beanutils library as the only * classes which use this class are BeanUtilsBean and ConvertUtilsBean and - * there is no obvious reason for a user of the beanutils library to subclass + * there is no obvious reason for a user of the BeanUtils library to subclass * either of those classes.</p> * * <p><strong>Note:</strong> A WeakHashMap bug in several 1.3 JVMs results in diff --git a/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java b/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java index 41f96440..3cdbab65 100644 --- a/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java +++ b/src/main/java/org/apache/commons/beanutils2/PropertyUtilsBean.java @@ -1941,8 +1941,8 @@ public class PropertyUtilsBean { * or an Array).</li> * </ul> * <p> - * The default behavior of beanutils 1.7.1 or later is for assigning to - * "a.b" to mean a.put(b, obj) always. However the behavior of beanutils + * The default behavior of BeanUtils 1.7.1 or later is for assigning to + * "a.b" to mean a.put(b, obj) always. However the behavior of BeanUtils * version 1.6.0, 1.6.1, 1.7.0 was for "a.b" to mean a.setB(obj) if such * a method existed, and a.put(b, obj) otherwise. In version 1.5 it meant * a.put(b, obj) always (ie the same as the behavior in the current version). diff --git a/src/main/java/org/apache/commons/beanutils2/locale/BaseLocaleConverter.java b/src/main/java/org/apache/commons/beanutils2/locale/BaseLocaleConverter.java index 55def0d8..d18c715f 100644 --- a/src/main/java/org/apache/commons/beanutils2/locale/BaseLocaleConverter.java +++ b/src/main/java/org/apache/commons/beanutils2/locale/BaseLocaleConverter.java @@ -222,7 +222,7 @@ public abstract class BaseLocaleConverter<T> implements LocaleConverter<T> { if (useDefault) { return getDefaultAs(targetType); } - // symmetric beanutils function allows null + // symmetric BeanUtils function allows null // so do not: throw new ConversionException("No value specified"); LOG.debug("Null value specified for conversion, returning null"); return null; diff --git a/src/main/java/org/apache/commons/beanutils2/locale/package-info.java b/src/main/java/org/apache/commons/beanutils2/locale/package-info.java index 0e554ea0..bfafd79c 100644 --- a/src/main/java/org/apache/commons/beanutils2/locale/package-info.java +++ b/src/main/java/org/apache/commons/beanutils2/locale/package-info.java @@ -16,7 +16,7 @@ */ /** - * Locale-aware extensions of the standard beanutils classes. + * Locale-aware extensions of the standard BeanUtils classes. * This package allows locale-dependent population of JavaBeans. */ package org.apache.commons.beanutils2.locale; diff --git a/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java b/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java index b7e3d8a9..228689b7 100644 --- a/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java +++ b/src/test/java/org/apache/commons/beanutils2/BeanificationTestCase.java @@ -134,7 +134,7 @@ public class BeanificationTestCase extends TestCase { // No action required } - /** Tests whether different threads can set beanutils instances correctly */ + /** Tests whether different threads can set BeanUtils instances correctly */ public void testBeanUtilsBeanSetInstance() throws Exception { final class SetInstanceTesterThread extends Thread { @@ -411,9 +411,9 @@ public class BeanificationTestCase extends TestCase { } } - /** Tests whether class loaders and beans are released from memory by the map used by beanutils */ + /** Tests whether class loaders and beans are released from memory by the map used by BeanUtils. */ public void testMemoryLeak2() throws Exception { - // tests when the map used by beanutils has the right behavior + // tests when the map used by BeanUtils has the right behavior if (BeanUtilsBeanTestCase.isPre14JVM()) { System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM"); diff --git a/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java b/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java index 64b5f41c..b87f5f03 100644 --- a/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java +++ b/src/test/java/org/apache/commons/beanutils2/PropertyUtilsTestCase.java @@ -2457,7 +2457,7 @@ public class PropertyUtilsTestCase extends TestCase { // probably just tried to do // map.set("submap(beta2)", betaBean2) // which is almost certainly not what the used expected. This is - // what beanutils 1.5.0 to 1.7.1 did.... + // what BeanUtils 1.5.0 to 1.7.1 did.... fail("Exception not thrown for invalid setNestedProperty syntax"); } catch (final IllegalArgumentException ex) { // ok, getting an exception was expected. As it is of a generic @@ -2480,7 +2480,7 @@ public class PropertyUtilsTestCase extends TestCase { // probably just tried to do // map.get("submap[3]") // which is almost certainly not what the used expected. This is - // what beanutils 1.5.0 to 1.7.1 did.... + // what BeanUtils 1.5.0 to 1.7.1 did.... fail("Exception not thrown for invalid setNestedProperty syntax"); } catch (final IllegalArgumentException ex) { // ok, getting an exception was expected. As it is of a generic diff --git a/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java b/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java index 5b76178f..9638d18d 100644 --- a/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java +++ b/src/test/java/org/apache/commons/beanutils2/locale/LocaleBeanificationTestCase.java @@ -42,7 +42,7 @@ import org.junit.jupiter.api.Test; /** * <p> - * Test Case for changes made during LocaleBeanutils Beanification. This is basically a cut-and-correct version of the beanutils beanifications tests. + * Test Case for changes made during LocaleBeanutils Beanification. This is basically a cut-and-correct version of the BeanUtils beanifications tests. * </p> */ public class LocaleBeanificationTestCase { @@ -113,7 +113,7 @@ public class LocaleBeanificationTestCase { LocaleConvertUtils.deregister(); } - /** Tests whether different threads can set beanutils instances correctly */ + /** Tests whether different threads can set BeanUtils instances correctly */ @Test public void testBeanUtilsBeanSetInstance() throws Exception { @@ -444,9 +444,9 @@ public class LocaleBeanificationTestCase { } } - /** Tests whether class loaders and beans are released from memory by the map used by beanutils */ + /** Tests whether class loaders and beans are released from memory by the map used by BeanUtils */ public void testMemoryLeak2() { - // tests when the map used by beanutils has the right behavior + // tests when the map used by BeanUtils has the right behavior if (BeanUtilsBeanTestCase.isPre14JVM()) { System.out.println("WARNING: CANNOT TEST MEMORY LEAK ON PRE1.4 JVM");