[lworld] RFR: 8372515: [lworld] Plumb in javac flags for compiling with preview mode [v2]
Maurizio Cimadamore
mcimadamore at openjdk.org
Wed Dec 10 11:13:53 UTC 2025
On Tue, 25 Nov 2025 15:32:51 GMT, David Beaumont <duke at openjdk.org> wrote:
>> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove note about StableValue (not possible)
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/file/JRTIndex.java line 147:
>
>> 145:
>> 146: synchronized Entry getEntry(RelativeDirectory rd) throws IOException {
>> 147: if (isClosed) {
>
> One other example of a new exception, not previously possible. This is rather unavoidable post-closure, and feels like IOException is the appropriate response (as opposed to the IllegalStateException proposed in close()).
I'm not too sure. It seems a case of calling an operation when the object is in a bad state. So, ISE or an assertion error seems also legitimate.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1761#discussion_r2606227492
More information about the valhalla-dev
mailing list