RFR: JDK-8073187: Unexpected side effect in Pack200

Xueming Shen xueming.shen at oracle.com
Wed Sep 30 22:57:57 UTC 2015


looks good!

sherman

On 09/23/2015 10:42 AM, Kumar Srinivasan wrote:
> Hi John, Sherman,
>
> I noticed a flaw in my fix, ie. in a section of the logic that is present
> but unused,  and reserved for future use, fixed it here anyway, for
> correctness.
>
> The full webrev:
> http://cr.openjdk.java.net/~ksrini/8073187/webrev.1/
>
> The delta changes:
> http://cr.openjdk.java.net/~ksrini/8073187/webrev.1/webrev.delta/index.html
>
> Thanks
> Kumar
>
>> Hello,
>>
>> Pack200 converts the timestamps to UTC for the pack200 archive format,
>>  in order to do so,  the default TimeZone was set to UTC, while reading and
>> writing archives. This is problematic in a multi threaded environment.
>>
>> With the fix for JDK-8075526 in place, the pack200 code is refactored
>> to use the new Jar/Zip APIs, and eliminates issues arising from pack200
>> setting the default TimeZone.
>>
>> http://cr.openjdk.java.net/~ksrini/8073187/webrev.0/
>>
>> Thanks
>> Kumar
>>
>




More information about the core-libs-dev mailing list