RFR: 8255968: Confusing error message for inaccessible constructor [v3]

Guoxiong Li github.com+13688759+lgxbslgx at openjdk.java.net
Wed Nov 25 07:03:14 UTC 2020


> Hi all,
> 
> When using inaccessible constructor, compiler would output some confusing error message occasionally. These confusing error message is related to the constructor order. See [JDK-8255968](https://bugs.openjdk.java.net/browse/JDK-8255968) for more information.
> This patch solves this bug, regardless of the construction order, compiler can always output expected message.
> Thank you for taking the time to review.
> 
> Best Regards.

Guoxiong Li has updated the pull request incrementally with three additional commits since the last revision:

 - Change the `Test(String x)` from private to default. Add comments in test cases to improve readability.
 - Add corresponding tests for the method reference cases.
 - Add comments to state the situations

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1389/files
  - new: https://git.openjdk.java.net/jdk/pull/1389/files/a9a68149..98ee2f2a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1389&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1389&range=01-02

  Stats: 268 lines in 23 files changed: 250 ins; 4 del; 14 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1389.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1389/head:pull/1389

PR: https://git.openjdk.java.net/jdk/pull/1389


More information about the compiler-dev mailing list