On 27/07/2020 21:42, Joe Darcy wrote:
Hello,
Another module, another set of default constructors to replace with
explicit ones. Please review the code changes and CSR to address:
JDK-8250640: Address reliance on default constructors in jdk.jdi
webrev: http://cr.openjdk.java.net/~darcy/8250640.0/
CSR: https://bugs.openjdk.java.net/browse/JDK-8250642
Patch below; I'll update copyrights before pushing. In the Bootstrap
class, since it doesn't define any non-static methods, it looks like
it doesn't need a constructor in its API at all so I terminally
deprecated the constructor.
Right, Bootstrap should have a public constructor. Ideally the abstract
classes in the spi package would have a protected constructor.
The change looks okay to me.
-Alan
