<div dir="ltr"><div>Hello Per and Leyden subscribers,<br>First, I am glad that we are finally adding an API that exposes one of core libraries' favorite feature, `@Stable`, to common users!<br><br>For the API design, however, I have a request: Can we have a ComputedConstant factory that creates a List<V> in addition to one that creates a List<ComputedConstant<V>>?<br><br>I think using the List<ComputedConstant<V>> is confusing. The example usages in the JEP [1] and in the API specification [2] are already wrong: we need an extra ConputedConstant.get() call to unwrap the ComputedConstant after List.get(index) call, which currently returns a computed constant than the actual value.<br><br>The current List<ComputedConstant<V>> is to be kept in case users want fine-grained control over each constant's resolution failure, etc. and covers the new factory's functionality. But I believe the new factory will see wider usage:<br><br>1. None of the 2 old patterns in the "Motivation" section uses any of these exception handling or initialization state detection.<br>2. Returning a List<V> allows users to conveniently pass the list in usages instead of using streams or writing custom wrappers.<br><br>A follow up to a previous request [3], I believe having a map (of type Map<K, V> instead of Map<K, ComputedConstant<V>>) would be feasible too.<br><br>Finally, a side comment about the current OnDemandComputedConstantList: it computes ComputedConstant wrappers in addition to the actual constants on demand, which... seems a bit overkill, when ComputedConstant itself is already a lightweight wrapper of a heavy computation?<br><br>Best,<br>Chen Liang<br><br>[1]: <a href="https://openjdk.org/jeps/8312611">https://openjdk.org/jeps/8312611</a>  "var kbd = lbl.labels.get(3);"<br>[2]: <a href="https://cr.openjdk.org/~pminborg/computed-constant/api/java.base/java/lang/ComputedConstant.html#of(int,java.util.function.IntFunction)">https://cr.openjdk.org/~pminborg/computed-constant/api/java.base/java/lang/ComputedConstant.html#of(int,java.util.function.IntFunction)</a>  "return PO2_CACHE.get(n);"<br>[3]: <a href="https://mail.openjdk.org/pipermail/leyden-dev/2023-August/000277.html">https://mail.openjdk.org/pipermail/leyden-dev/2023-August/000277.html</a><br></div></div>