RFR 4947890 : Minimize JNI upcalls in system property initialization
Brent Christian
brent.christian at oracle.com
Mon Nov 26 23:12:04 UTC 2018
Hi,
On 11/19/18 3:37 PM, Roger Riggs wrote:
>> Raw::xxx_NDX are initialized to 1 + previous_NDX. It's a general
>> good approach to increment the index but I find it error-prone and
>> hard to catch mistake since the (adjacent) variable names look
>> so alike. Perhaps some form of verification or assertion to ensure
>> the indices are correctly initialized.
>
> Added a test that uses reflection to verify the uniqueness and sequence.
The test says:
* Check that the Raw._*_NDX indexes are sequential
and followed by the FIXED_LENGTH value.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
though the code does not verify that FIXED_LENGTH is the final value. I
don't know that it's all that important to do, but I thought I'd point
it out.
Thanks,
-Brent
More information about the build-dev
mailing list