Integrated: 8267578: Remove unnecessary preview checks

[Guoxiong Li]

On Sun, 23 May 2021 11:04:56 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> The following preview checks are unnecessary because they are standard features now.
> 
> Attr.java
> 
>         allowReifiableTypesInInstanceof =
>                 Feature.REIFIABLE_TYPES_INSTANCEOF.allowedInSource(source) &&
>                 (!preview.isPreview(Feature.REIFIABLE_TYPES_INSTANCEOF) || preview.isEnabled());
> 
> 
> JavacParser.java
> 
>         this.allowYieldStatement = (!preview.isPreview(Feature.SWITCH_EXPRESSION) || preview.isEnabled()) &&
>                 Feature.SWITCH_EXPRESSION.allowedInSource(source);
> 
> 
> Resolve.java
> 
>         allowYieldStatement = (!preview.isPreview(Feature.SWITCH_EXPRESSION) || preview.isEnabled()) &&
>                 Feature.SWITCH_EXPRESSION.allowedInSource(source);
> 
> 
> It is good to remove them.
> Thanks for review.
> 
> Best Regards,
> -- Guoxiong

This pull request has now been integrated.

Changeset: f4531661
Author:    Guoxiong Li <gli at openjdk.org>
Committer: Jan Lahoda <jlahoda at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f4531661b55458554f907ba988eca22b72c72bf3
Stats:     10 lines in 3 files changed: 0 ins; 7 del; 3 mod

8267578: Remove unnecessary preview checks

Reviewed-by: jlahoda

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

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