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.


More information about the core-libs-dev mailing list