RFR: 8345248: Module name 'transitive' not accepted for `requires transitive` [v2]

Jan Lahoda jlahoda at openjdk.org
Fri Nov 29 14:30:57 UTC 2024


> Considering module named `transitive`, it should be possible to write: `requires transitive;`, which works, and also `requires transitive transitive;`, which currently does not work:
> 
> $ javac -d out/ --module-source-path . transitive/module-info.java test/module-info.java
> test/module-info.java:2: error: repeated modifier
>     requires transitive transitive;
>                         ^
> test/module-info.java:2: error: <identifier> expected
>     requires transitive transitive;
>                                   ^
> 2 errors
> 
> 
> The proposed solution is to properly handle lookahead if the transitive flag has already been seen.

Jan Lahoda has updated the pull request incrementally with one additional commit since the last revision:

  Fixing test.

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/22460/files
  - new: https://git.openjdk.org/jdk/pull/22460/files/11932dcc..0a0e6d30

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22460&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22460&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22460.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22460/head:pull/22460

PR: https://git.openjdk.org/jdk/pull/22460


More information about the compiler-dev mailing list