[lworld] RFR: 8373806: [lworld] Disable preview patching by default [v3]
Roger Riggs
rriggs at openjdk.org
Tue Dec 16 18:13:52 UTC 2025
On Tue, 16 Dec 2025 17:08:31 GMT, David Beaumont <duke at openjdk.org> wrote:
>> Flip the 3 places where the PREVIEW_PATCHING_DEFAULT property default is defined.
>>
>> This will enable the JRT file-system preview mode code for *almost* all tests and code by default.
>> A few tests still set the property explicitly to false, but everything else should now no longer be using preview patching.
>
> David Beaumont has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 14 commits:
>
> - Merge branch 'plumbing' into flip_default
> - Merge branch 'lworld' into plumbing
> - flip default property value
> - Merge branch 'lworld' into plumbing
> - correct assertion
> - review feedback
> - remove automatic formatting
> - comments
> - Remove note about StableValue (not possible)
> - Use byte array not stream for simplicity
> - ... and 4 more: https://git.openjdk.org/valhalla/compare/7bcb2999...e7924de7
It doesn't seem necessary to merge in the "plumb"ing PR. that should have come from lworld when the plumbing was integrated and the dependency updated by skara.
There should only be the 1 commit with the "true" enables.
But at least it only shows the correct three files with the changes.
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/1806#issuecomment-3661799260
More information about the valhalla-dev
mailing list