RFR: 8346048: test/lib/containers/docker/DockerRunOptions.java uses addJavaOpts() from ctor

Leonid Mesnik lmesnik at openjdk.org
Mon Dec 16 02:41:28 UTC 2024


On Sat, 14 Dec 2024 00:21:38 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> Trivial fix to avoid using methods from DockerRunOptions().
> 
> Don't methods final, just  for using them in ctor.

Yes, should be mentioned that it is just to fix javac warning. I am not sure if anyone going extend this class, so let make the methods final. If anyone wants to override them they should deal with javac warnings/errors/

-------------

PR Comment: https://git.openjdk.org/jdk/pull/22748#issuecomment-2544417820


More information about the hotspot-runtime-dev mailing list