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