[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