[lworld] RFR: 8373896: [lworld] Remove now-dead preview patching code
David Beaumont
duke at openjdk.org
Wed Dec 17 17:51:14 UTC 2025
On Wed, 17 Dec 2025 16:55:15 GMT, David Beaumont <duke at openjdk.org> wrote:
> Removing code guarded by now obsolete preview patching property.
> This should be a no-op functionally.
> Some minor tidying around arguments.cpp since now there's no reason to init the jimage file in the module patching code.
test/jdk/tools/jlink/runtimeImage/PackagedModulesVsRuntimeImageLinkTest.java line 58:
> 56: * @build tests.* jdk.test.lib.process.OutputAnalyzer
> 57: * jdk.test.lib.process.ProcessTools
> 58: * @run main/othervm/timeout=1200 -ea -esa -DDISABLE_PREVIEW_PATCHING=false -Xmx1g PackagedModulesVsRuntimeImageLinkTest
Not 100% sure why this was explicitly leaving patching on, but it passes locally.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1819#discussion_r2627919939
More information about the valhalla-dev
mailing list