Warnings Cleanup in java.util. (more from hack day) (original) (raw)

Martijn Verburg martijnverburg at gmail.com
Fri Dec 2 14:32:02 PST 2011

Hi Sherman,

I think the original patch provider (Prasanna) was aiming at simply removing the deprecation warning. Interesting point on the performance aspect!

Is performance for this class already measured in an OpenJDK test somewhere? Perhaps we could provide a test suite in this instance?

Cheers, Martijn

On Friday, 2 December 2011, Xueming Shen <xueming.shen at oracle.com> wrote:

The j.u.z.ZipEntry change should NOT be in as a warning cleanup. Will need further evaluation/measurement, for example from performance point of view, the change now creates a new Calendar instance instead of a Date object (Date class uses a singleton BaseCalendar for the calculation, if I remember correctly) for each invocation. -Sherman On 12/02/2011 11:24 AM, Michael Barker wrote:

Hi, We had some late submissions from our hack day participants. 2 more patches. One (j.u.z.ZipEntry) is possibly a little more involved for a warnings fix. It removes a usage of deprecated Date APIs in favour of Calendar. However, the fix results in code that is very similar to Apache Harmony implementation. The other is a couple of fixes for generics (j.u.j.Manifest). Thanks to: - Prasannaa Regards, Michael Barker mikeb2701

More information about the jdk8-dev mailing list