[9] RFR: 8176041: Optimize handling of comment lines in Properties$LineReader.readLine

Claes Redestad claes.redestad at oracle.com
Thu Mar 2 11:35:12 UTC 2017


Hi,

as this now significantly helps with one of our startup regression, and
the change is very low risk, I've retargetted to 9.  Any objections?

/Claes

On 03/01/2017 05:17 PM, Aleksey Shipilev wrote:
> On 03/01/2017 06:11 PM, Claes Redestad wrote:
>> Sure, with a bit of fix-up this drops another 170k executed bytecodes
>> to read in the java.security file (original: 1882k, now: 980k):
>>
>> http://cr.openjdk.java.net/~redestad/8176041/jdk.02/
> Ok, that's fine.
>
> This is slightly more idiomatic, you might want to change before pushing:
>    c = (char)(b & 0xFF);
>
> Thanks,
> -Aleksey
>



More information about the core-libs-dev mailing list