Warning Fixes from LJC Hack Session
Weijun Wang
weijun.wang at oracle.com
Thu Feb 9 19:40:19 PST 2012
Just give me anything that includes the jars not found yet. I just want
to extract the hash line and cert info from them to make sure they can
be removed from trusted.libraries safely.
Thanks
Max
On 02/10/2012 11:11 AM, Stuart Marks wrote:
> Hi Mike,
>
> I finally got back to this. These fixes look pretty good and knock off
> 100+ additional warnings! I've filed bug 7143230 [1] to track this. I
> took a look through the code and I took the liberty of fixing up a few
> very minor things:
>
> 1. Removed unnecessary casts to ZipEntry in JarVerifier.java, suggested
> by Chirs Hegarty [2]. (These aren't strictly redundant casts, and don't
> cause warnings, as the origin types are <? extends ZipEntry> and
> JarEntry. However, they are unnecessary.)
>
> 2. Fixed typo in unrelated comment at line 50 in SignatureFile.java that
> I happened to notice.
>
> 3. Removed parentheses from expressions in MemoryMonitor.java lines 216,
> 219 which are now unnecessary since the cast has been removed.
>
> No need to issue another patch; I'll just include these changes when I
> push the changeset.
>
> Which brings me to the topic that we discussed before when I pushed
> LJC's previous round of warnings fixes, that is, how the Contributed-by
> line in the commit message should be formatted. (See [3] for the
> requirements.) For reference, here's what the changeset comment for the
> previous set of LJC fixes ended up looking like:
>
>
> changeset: 4802:4f0f9f9c4892
> user: smarks
> date: Wed Dec 07 12:12:50 2011 -0800
> description:
> 7117249: fix warnings in java.util.jar, .logging, .prefs, .zip
> Reviewed-by: alanb, dholmes, forax, sherman, smarks
> Contributed-by: Prasannaa <prasannaa_ss at yahoo.com>, Martijn Verburg
> <martijnverburg at gmail.com>, Goerge_Albrecht <goerge.albrecht at gmx.net>,
> Graham Allan <grundlefleck at googlemail.com>, Michael Barker
> <mikeb01 at gmail.com>
>
>
> It looks like a different set of people contributed to this round of
> fixes. If you could send me the list of names and email addresses, I can
> format them into the commit message and push the fix.
>
> Thanks!
>
> s'marks
>
>
> [1] http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7143230
>
> [2]
> http://mail.openjdk.java.net/pipermail/jdk8-dev/2012-February/000715.html
>
> [3] http://openjdk.java.net/guide/producingChangeset.html
>
> On 2/4/12 12:12 AM, Chris Hegarty wrote:
>> Thanks for this, looks great.
>>
>> Good to see JarVerifier getting some much needed TLC.
>>
>> -Chris.
>>
>>
>> On 02/ 4/12 07:50 AM, Michael Barker wrote:
>>>> I see Rémi has suggested a slice& dice but I think that's a bit too
>>>> much
>>>> work for the changes involved. Instead I would suggest a simple
>>>> split, send
>>>> the AWT/Printing/Beans changes to awt-dev + 2d-dev, and everything
>>>> else to
>>>> core-libs-dev.
>>>
>>> Attached is the patch that contains "everthing else" from LJC warning
>>> fixes hack session.
>>>
>>> Mike.
More information about the jdk8-dev
mailing list