RFR: 8254274: Development to add 'lint' warning for @ValueBased classes [v3]

Srikanth Adayapalam sadayapalam at openjdk.java.net
Thu Oct 29 13:23:51 UTC 2020


On Thu, 29 Oct 2020 10:24:58 GMT, Jim Laskey <jlaskey at openjdk.org> wrote:

> Wondering if keyword SYNCHRONIZATION is too broad, suggest VALUE_ SYNCHRONIZATION. Would it make sense to have a test for a class.ref type value?

class.ref is a valhalla'ism - but the present work is targetting JDK16 as a precursor/preparation for Valhalla. (even though the integration first happens to jep390 branch of Valhalla repo as a temporary home). So the .ref syntax is really not available to test/need not be tested.

As for the keyword, I am comfortable with it, but I will make it a point to solicit input at the next stage of review. (The accumulated cross component changes will get to a RFR when targetting jdk16)

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

PR: https://git.openjdk.java.net/valhalla/pull/237



More information about the valhalla-dev mailing list