On Fri, 3 Nov 2023 15:29:25 GMT, Jim Laskey <jlas...@openjdk.org> wrote:

>> Update String Templates for a second preview. With the addition of
>> 
>> - Expression type and throws are determined from the `process` method of the 
>> processor type and not the processor type.
>> 
>> - Qualified `STR` and `RAW` are treated the same as unqualified `STR` and 
>> `RAW` .
>> 
>> - Raw (generic) process types are no longer an error.
>
> Jim Laskey has updated the pull request with a new target base due to a merge 
> or a rebase. The incremental webrev excludes the unrelated changes brought in 
> by the merge/rebase. The pull request contains six additional commits since 
> the last revision:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8315457
>  - Cache process method type in JCStringTemplate
>  - Revert source
>  - Revert @since 22
>  - Accept qualified STR and RAW
>  - String Templates (second preview)

Marked as reviewed by alanb (Reviewer).

src/java.base/share/classes/jdk/internal/access/JavaLangAccess.java line 424:

> 422:     * @since 21
> 423:     */
> 424:    @PreviewFeature(feature=PreviewFeature.Feature.STRING_TEMPLATES)

I think you can drop both `@since` and `@PreviewFeature` from these methods. 
This in an internal interface, used for shared secrets, there is a lot of churn 
in JLA in each release.

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

PR Review: https://git.openjdk.org/jdk/pull/16202#pullrequestreview-1713768565
PR Review Comment: https://git.openjdk.org/jdk/pull/16202#discussion_r1382392709

Reply via email to