JVM: Don't use deprecated Class#newInstance()

Author: sideeffffect

jvm/src/main/scala/org/portablescala/reflect/InstantiatableClass.scala

@@ -1,26 +1,35 @@
 package org.portablescala.reflect
 
-/** A wrapper for a class that can be instantiated.
+import java.lang.reflect.InvocationTargetException
+
+/**
+ * A wrapper for a class that can be instantiated.
  *
  *  @param runtimeClass
  *    The `java.lang.Class[_]` representing the class.
  */
 final class InstantiatableClass private[reflect] (val runtimeClass: Class[_]) {
+
   /** A list of the public constructors declared in this class. */
   val declaredConstructors: List[InvokableConstructor] =
-    runtimeClass.getConstructors().map(new InvokableConstructor(_)).toList
+    runtimeClass.getConstructors.map(new InvokableConstructor(_)).toList
 
-  /** Instantiates this class using its zero-argument constructor.
+  /**
+   * Instantiates this class using its zero-argument constructor.
    *
    *  @throws java.lang.InstantiationException
    *    (caused by a `NoSuchMethodException`)
    *    If this class does not have a public zero-argument constructor.
    */
   def newInstance(): Any = {
     try {
-      runtimeClass.newInstance()
+      runtimeClass.getDeclaredConstructor().newInstance()
     } catch {
-      case e: IllegalAccessException =>
+      case e: InvocationTargetException if e.getCause != null =>
+        throw e.getCause
+      case e: NoSuchMethodException =>
+        throw new InstantiationException(runtimeClass.getName).initCause(e)
+      case _: IllegalAccessException =>
         /* The constructor exists but is private; make it look like it does not
          * exist at all.
          */