You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2023/10/02 15:07:05 UTC

[commons-beanutils] branch 1.X updated: Update call deprecated in Java 9

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch 1.X
in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git


The following commit(s) were added to refs/heads/1.X by this push:
     new adc93776 Update call deprecated in Java 9
adc93776 is described below

commit adc937763620d18a4a8818bfd3580ea954675f11
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Mon Oct 2 11:07:00 2023 -0400

    Update call deprecated in Java 9
---
 src/main/java/org/apache/commons/beanutils/BeanMap.java  |  4 ++--
 .../java/org/apache/commons/beanutils/BeanUtilsBean.java |  2 +-
 .../java/org/apache/commons/beanutils/LazyDynaBean.java  |  6 +++---
 .../java/org/apache/commons/beanutils/LazyDynaList.java  |  2 +-
 .../java/org/apache/commons/beanutils/LazyDynaMap.java   |  2 +-
 .../apache/commons/beanutils/bugs/Jira347TestCase.java   |  2 +-
 .../beanutils/converters/ClassReloaderTestCase.java      |  4 ++--
 .../commons/beanutils/converters/MemoryTestCase.java     |  2 +-
 .../beanutils/memoryleaktests/MemoryLeakTestCase.java    | 16 ++++++++--------
 9 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/main/java/org/apache/commons/beanutils/BeanMap.java b/src/main/java/org/apache/commons/beanutils/BeanMap.java
index 51e543dc..4af2e53f 100644
--- a/src/main/java/org/apache/commons/beanutils/BeanMap.java
+++ b/src/main/java/org/apache/commons/beanutils/BeanMap.java
@@ -270,7 +270,7 @@ public class BeanMap extends AbstractMap<Object, Object> implements Cloneable {
         Object newBean = null;
         final Class<? extends Object> beanClass = bean.getClass(); // Cannot throw Exception
         try {
-            newBean = beanClass.newInstance();
+            newBean = beanClass.getConstructor().newInstance();
         } catch (final Exception e) {
             // unable to instantiate
             final CloneNotSupportedException cnse = new CloneNotSupportedException
@@ -346,7 +346,7 @@ public class BeanMap extends AbstractMap<Object, Object> implements Cloneable {
         Class<? extends Object> beanClass = null;
         try {
             beanClass = bean.getClass();
-            bean = beanClass.newInstance();
+            bean = beanClass.getConstructor().newInstance();
         }
         catch (final Exception e) {
             final UnsupportedOperationException uoe =
diff --git a/src/main/java/org/apache/commons/beanutils/BeanUtilsBean.java b/src/main/java/org/apache/commons/beanutils/BeanUtilsBean.java
index fbdf9270..09ca5f62 100644
--- a/src/main/java/org/apache/commons/beanutils/BeanUtilsBean.java
+++ b/src/main/java/org/apache/commons/beanutils/BeanUtilsBean.java
@@ -177,7 +177,7 @@ public class BeanUtilsBean {
         if (bean instanceof DynaBean) {
             newBean = ((DynaBean) bean).getDynaClass().newInstance();
         } else {
-            newBean = bean.getClass().newInstance();
+            newBean = bean.getClass().getConstructor().newInstance();
         }
         getPropertyUtils().copyProperties(newBean, bean);
         return (newBean);
diff --git a/src/main/java/org/apache/commons/beanutils/LazyDynaBean.java b/src/main/java/org/apache/commons/beanutils/LazyDynaBean.java
index 28179712..bd34e512 100644
--- a/src/main/java/org/apache/commons/beanutils/LazyDynaBean.java
+++ b/src/main/java/org/apache/commons/beanutils/LazyDynaBean.java
@@ -701,7 +701,7 @@ public class LazyDynaBean implements DynaBean, Serializable {
                 indexedProperty = defaultIndexedProperty(name);
             } else {
                 try {
-                    indexedProperty = type.newInstance();
+                    indexedProperty = type.getConstructor().newInstance();
                 }
                 catch (final Exception ex) {
                     throw new IllegalArgumentException
@@ -740,7 +740,7 @@ public class LazyDynaBean implements DynaBean, Serializable {
 
         } else if (Map.class.isAssignableFrom(type)) {
             try {
-                mappedProperty = type.newInstance();
+                mappedProperty = type.getConstructor().newInstance();
             }
             catch (final Exception ex) {
                 throw new IllegalArgumentException
@@ -837,7 +837,7 @@ public class LazyDynaBean implements DynaBean, Serializable {
         }
 
         try {
-            return type.newInstance();
+            return type.getConstructor().newInstance();
         }
         catch (final Exception ex) {
             if (logger().isWarnEnabled()) {
diff --git a/src/main/java/org/apache/commons/beanutils/LazyDynaList.java b/src/main/java/org/apache/commons/beanutils/LazyDynaList.java
index d1292a3e..d4dbcdf3 100644
--- a/src/main/java/org/apache/commons/beanutils/LazyDynaList.java
+++ b/src/main/java/org/apache/commons/beanutils/LazyDynaList.java
@@ -523,7 +523,7 @@ public class LazyDynaList extends ArrayList<Object> {
         // Create a new object of the specified type
         Object object = null;
         try {
-            object = elementType.newInstance();
+            object = elementType.getConstructor().newInstance();
         } catch (final Exception e) {
             throw new IllegalArgumentException("Error creating type: "
                            + elementType.getName() + " - " + e);
diff --git a/src/main/java/org/apache/commons/beanutils/LazyDynaMap.java b/src/main/java/org/apache/commons/beanutils/LazyDynaMap.java
index b99901a0..277933c1 100644
--- a/src/main/java/org/apache/commons/beanutils/LazyDynaMap.java
+++ b/src/main/java/org/apache/commons/beanutils/LazyDynaMap.java
@@ -275,7 +275,7 @@ public class LazyDynaMap extends LazyDynaBean implements MutableDynaClass {
             @SuppressWarnings("unchecked")
             final
             // The new map is used as properties map
-            Map<String, Object> temp = getMap().getClass().newInstance();
+            Map<String, Object> temp = getMap().getClass().getConstructor().newInstance();
             newMap = temp;
         } catch(final Exception ex) {
             newMap = newMap();
diff --git a/src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java b/src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java
index 36f833a4..1bf2ae6f 100644
--- a/src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java
+++ b/src/test/java/org/apache/commons/beanutils/bugs/Jira347TestCase.java
@@ -53,7 +53,7 @@ public class Jira347TestCase extends TestCase {
         final String className = "org.apache.commons.beanutils.MappedPropertyTestBean";
         final ClassLoader loader = newClassLoader();
         final Class<?> beanClass    = loader.loadClass(className);
-        beanClass.newInstance();
+        beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         // Sanity checks only
diff --git a/src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java b/src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java
index 2080043e..8008c77d 100644
--- a/src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils/converters/ClassReloaderTestCase.java
@@ -63,8 +63,8 @@ public class ClassReloaderTestCase extends TestCase {
         assertTrue(sharedLoader != componentLoader);
 
         // verify that objects of these two types are not assignment-compatible
-        final Object obj1 = sharedClass.newInstance();
-        final Object obj2 = componentClass.newInstance();
+        final Object obj1 = sharedClass.getConstructor().newInstance();
+        final Object obj2 = componentClass.getConstructor().newInstance();
 
         assertTrue("Obj1 class incorrect", sharedClass.isInstance(obj1));
         assertFalse("Obj1 class incorrect", componentClass.isInstance(obj1));
diff --git a/src/test/java/org/apache/commons/beanutils/converters/MemoryTestCase.java b/src/test/java/org/apache/commons/beanutils/converters/MemoryTestCase.java
index 23725c2f..b9c05d2a 100644
--- a/src/test/java/org/apache/commons/beanutils/converters/MemoryTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils/converters/MemoryTestCase.java
@@ -183,7 +183,7 @@ public class MemoryTestCase {
               // a class FloatConverter has been loaded from the component's
               // private classpath.
               final Class<?> newFloatConverterClass = componentLoader.reload(FloatConverter.class);
-              Object newFloatConverter = newFloatConverterClass.newInstance();
+              Object newFloatConverter = newFloatConverterClass.getConstructor().newInstance();
               assertTrue(newFloatConverter.getClass().getClassLoader() == componentLoader);
 
               // verify that this new object does implement the Converter type
diff --git a/src/test/java/org/apache/commons/beanutils/memoryleaktests/MemoryLeakTestCase.java b/src/test/java/org/apache/commons/beanutils/memoryleaktests/MemoryLeakTestCase.java
index 2e1d586e..d406d1f6 100644
--- a/src/test/java/org/apache/commons/beanutils/memoryleaktests/MemoryLeakTestCase.java
+++ b/src/test/java/org/apache/commons/beanutils/memoryleaktests/MemoryLeakTestCase.java
@@ -61,7 +61,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         final WeakReference<ClassLoader> someRef = new WeakReference<ClassLoader>(loader);
@@ -108,7 +108,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         final WeakReference<ClassLoader> someRef = new WeakReference<ClassLoader>(loader);
@@ -160,7 +160,7 @@ public class MemoryLeakTestCase {
         final String className = "org.apache.commons.beanutils.memoryleaktests.pojotests.SomeMappedPojo";
         final ClassLoader loader = newClassLoader();
         final Class<?> beanClass    = loader.loadClass(className);
-        final Object bean        = beanClass.newInstance();
+        final Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         // Sanity checks only
@@ -206,7 +206,7 @@ public class MemoryLeakTestCase {
         final String className = "org.apache.commons.beanutils.memoryleaktests.pojotests.SomeMappedPojo";
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         // Sanity checks only
@@ -258,7 +258,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         final WeakReference<ClassLoader> someRef = new WeakReference<ClassLoader>(loader);
@@ -305,7 +305,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         WrapDynaBean wrapDynaBean = new WrapDynaBean(bean);
         // -----------------------------------------------------------------------------
 
@@ -358,7 +358,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         final WeakReference<ClassLoader> someRef = new WeakReference<ClassLoader>(loader);
@@ -406,7 +406,7 @@ public class MemoryLeakTestCase {
         // The classLoader will go away only when these following variables are released
         ClassLoader loader = newClassLoader();
         Class<?> beanClass    = loader.loadClass(className);
-        Object bean        = beanClass.newInstance();
+        Object bean        = beanClass.getConstructor().newInstance();
         // -----------------------------------------------------------------------------
 
         final WeakReference<ClassLoader> someRef = new WeakReference<ClassLoader>(loader);