RFR: 8224202: Speed up Properties.load

Claes Redestad claes.redestad at oracle.com
Thu May 23 15:43:32 UTC 2019


Hi Roger,

On 2019-05-23 16:42, Roger Riggs wrote:
> Hi Claes,
> 
> Looking good.
> 
> Is there any performance benefit to using ?: instead of if..else at
> Line 509-515    c = (fromStream) ? (char) (byteBuf[off++] & 0xFF) : 
> charBuf[off++];
> 
> Line 578-581:   precedingBackslash = (c == '\\') ? !precedingBackslash : 
> false;

well, there's possibly a (tiny) benefit: ternary expressions generate
slightly more compact bytecode in some cases; in this case trims away
~40 bytes from LineReader.class. When looking at profiles of how many
get executed there's no real difference though, so it hardly matters.

> 
> Line 600:  add 'the"  to "characters in +the+ following line"

Done!

http://cr.openjdk.java.net/~redestad/8224202/open.04/

/Claes


More information about the core-libs-dev mailing list