On Mon, 31 Oct 2022 20:11:34 GMT, Jim Laskey <jlas...@openjdk.org> wrote:

>> Enhance the Java programming language with string templates, which are 
>> similar to string literals but contain embedded expressions. A string 
>> template is interpreted at run time by replacing each expression with the 
>> result of evaluating that expression, possibly after further validation and 
>> transformation. This is a [preview language feature and 
>> API](http://openjdk.java.net/jeps/12).
>
> Jim Laskey has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add @SafeVarargs declarations

src/java.base/share/classes/java/lang/template/TemplateRuntime.java line 132:

> 130:             return result;
> 131:         }
> 132:         for (Object value : st.values()) {

I think that valueTypes() should return the types of the values not the dynamic 
classes of the values.
Here there is no type information so it should be Object for all values.

It has also the nice property that the return type of the accessors returned by 
valueAccessors are the same as valueTypes() which is i believe more coherent.

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

PR: https://git.openjdk.org/jdk/pull/10889

Reply via email to