[10] RFR: 8184603: Create ObjectStreamField signature lazily when possible

Claes Redestad claes.redestad at oracle.com
Fri Jul 14 12:47:03 UTC 2017


On 2017-07-14 14:32, Chris Hegarty wrote:
>> On 14 Jul 2017, at 13:19, Claes Redestad <claes.redestad at oracle.com> wrote:
>>
>> Hi,
>>
>> as a small startup optimization, we could make some setup in ObjectStreamField
>> lazier by calculating the signature field at first use.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8184603
>> Webrev: http://cr.openjdk.java.net/~redestad/8184603/jdk.00/
> I think this is ok.  Should toString also use getSignature?

Yes, I thought I'd done that already.

>
> For extra points remove the superfluous null assignment of
> `signature` and `field` ( as well as `offset` ).

'field' is final so that'd make javac cry bloody murder, but I fixed the
rest and cleaned up a bit for consistency:

http://cr.openjdk.java.net/~redestad/8184603/jdk.01/

Thanks!

&Claes


More information about the core-libs-dev mailing list