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

Maurizio Cimadamore mcimadamore at openjdk.java.net
Fri Nov 27 18:35:55 UTC 2020


On Thu, 26 Nov 2020 15:18:48 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:

>>> @mcimadamore Do you agree with this style?
>> 
>> Yeah - of course nesting things is gonna mess up with access checks - sorry for sending you down the wrong direction!
>> 
>> You could prefix every class in the test with `T8255968_1` e.g.
>> 
>> class T8255968_1 {
>>     T8255968Test1 c = new T8255968_1_Test(0);
>> }
>> 
>> class T8255968_1_Test {
>>     private T8255968_1_Test(int x) {}
>> }
>
> I update the code about test cases. Thank you for taking the time to review.

I haven't forgotten about this - we're running some extensive corpus analysis to make sure that the patch won't introduce any unwanted regressions. Please bear with us :-)

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

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


More information about the compiler-dev mailing list