RFR: String Density/Compact String JEP 254 (update)
Alan Bateman
Alan.Bateman at oracle.com
Sun Nov 1 18:18:40 UTC 2015
On 30/10/2015 21:30, Xueming Shen wrote:
> Hi,
>
> Thanks for the comments/suggestions. Here are the updated webrevs with
> minor changes here
> and there based on the feedback.
I spent time going through the changes in jdk repo and it looks very good.
One thing that isn't clear to me is why StringUTF16 needs a native
method to test for big endian. Could it use Unsafe::isBigEndian or is
there a non-obvious bootstrapping/initialization issue here?
The additional test coverage looks quite good but I'm guessing many of
these tests overlap with existing tests. Are you planning to leave them
in the CompactStrings directory? I assume it would make more sense to
move them up to the String directory. I was also wondering if more of
the existing tests should be updated to run them both ways. In passing,
SerializationUtils has a semi-colon at L47 that looks a bit odd.
-Alan.
More information about the core-libs-dev
mailing list