portable-scala / portable-scala-reflect

Platform independent reflection for Scala
BSD 3-Clause "New" or "Revised" License
42 stars 14 forks source link

.newInstance is deprecated as of Java 9 #25

Closed LeeTibbert closed 2 years ago

LeeTibbert commented 3 years ago

This issue is a forget-me-not.

I accidentally built portable-scala-reflect using a Java version greater than 8. There was a deprecation warning about the use of the, indeed deprecated, .newInstance(). From a quick grep, it looked like that may be used in a number of places.

I hope to get back to fix this after my current series of PRs to add Scala Native support. I will not be offended if anybody gets this fixed before I can get back to it.

LeeTibbert commented 3 years ago

To check: ReadMe.md has an example using newInstance(). Is that the deprecated method or something valid.

sideeffffect commented 2 years ago

This has been fixed in https://github.com/portable-scala/portable-scala-reflect/pull/40

gzm0 commented 2 years ago

TY