RFR: 8335989: Implement JEP 494: Module Import Declarations (Second Preview) [v7]
Vicente Romero
vromero at openjdk.org
Wed Nov 13 20:43:44 UTC 2024
On Wed, 13 Nov 2024 12:33:16 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:
>> This is a current patch for module imports declarations, second preview. At least the JEP number and preview revision will need to be updated in `jdk.internal.javac.PreviewFeature.Feature`, but otherwise I believe this is ready to receive feedback.
>>
>> The main changes are:
>> - `requires transitive java.base;` is permitted, under the preview flag. Both javac and the runtime module system are updated to accept this directive when preview is enabled.
>> - the `java.se` module is using `requires transitive java.base;`, and is deemed to be participating in preview, so its classfile version is not tainted. Runtime is updated to access `requires transitive java.base;` in any `java.*`, considering all of them to be participating in preview. Can be tighten up to only `java.se` is desired.
>> - the types imported through module imports can be shadowed using on-demand imports. So, for example, having:
>>
>> import module java.base;
>> import module java.desktop;
>> ...
>> List l;//ambigous
>>
>> but:
>>
>> import module java.base;
>> import module java.desktop;
>> import java.util.*;
>> ...
>> List l;//not ambigous, reference to java.util.List
>
> Jan Lahoda has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 21 commits:
>
> - Merge branch 'master' into JDK-8335989
> - Merge branch 'master' into JDK-8335989
> - Moving operators to the beginning of line, as suggested.
> - Updating PreviewFeature metadata
> - Cleanup.
> - Merge branch 'master' into JDK-8335989
> - Merge branch 'master' into JDK-8335989
> - Reflecting review feedback.
> - Cleanup.
> - Cleanup.
> - ... and 11 more: https://git.openjdk.org/jdk/compare/79345bbb...babcfccf
looks good
-------------
Marked as reviewed by vromero (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21431#pullrequestreview-2434370098
More information about the core-libs-dev
mailing list