[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-compiler-dev
Subject:    Re: RFR: JDK-8235149: javac parser is too aggressive on ambiguous expressions using identifier: reco
From:       Maurizio Cimadamore <maurizio.cimadamore () oracle ! com>
Date:       2020-01-27 23:45:31
Message-ID: f393a2af-2e99-2ef2-04bb-12ca820e64c1 () oracle ! com
[Download RAW message or body]


On 27/01/2020 23:44, Vicente Romero wrote:
>
>
> On 1/27/20 6:30 PM, Maurizio Cimadamore wrote:
>> Looks good.
>>
>> Partially related question - is it normal/expected that we issue 
>> warning for a restricted type name even when preview features are 
>> disabled?
>
> in the case of records we decided to issue a warning for its use as a 
> type even if preview features were disabled. I think to remember that 
> we enabled the warning in JDK 13 to warn users in advance

Ok, thanks

Maurizio

>
>>
>> Maurizio
>
> Vicente
>
>>
>> On 27/01/2020 16:37, Vicente Romero wrote:
>>> Hi,
>>>
>>> Please review the fix for [1] at [2] which fixes a too aggressive 
>>> approach in the parser which is issuing errors for acceptable used 
>>> of the `record` identifier in cases when the preview feature flag is 
>>> not set.
>>>
>>> Thanks,
>>> Vicente
>>>
>>> PS, as part of the patch I have also removed a couple of debug 
>>> statements from RecordCompilationTests in an unrelated test.
>>>
>>> [1] https://bugs.openjdk.java.net/browse/JDK-8235149
>>> [2] http://cr.openjdk.java.net/~vromero/8235149/webrev.00/
>
[prev in list] [next in list] [prev in thread] [next in thread]