Code review request: 7064075 Security libraries don't build with javac -Xlint:all,-deprecation -Werror
Alexandre Boulgakov
alexandre.boulgakov at oracle.com
Mon Jul 18 21:35:44 UTC 2011
Is there an easy way to see when a class was added to the JDK?
Thanks,
Sasha
On 7/18/2011 1:43 PM, Sean Mullan wrote:
> On 7/18/11 1:17 PM, Alexandre Boulgakov wrote:
>> Please see my responses in-line.
>>
>> Thanks for reviewing!
>>
>> -Sasha
>>
>> On 7/15/2011 6:18 AM, Chris Hegarty wrote:
>>> On 07/15/11 01:19 PM, Sean Mullan wrote:
>>>> All the changes look good. I only have a couple of questions/comments:
>>>>
>>>> 1) How did you calculate the serialVersionUID for the classes that
>>>> had omitted
>>>> this field? Ideally you want to calculate it on a previous version of
>>>> the class,
>>>> for compatibility reasons. This is especially important for
>>>> Serializable objects
>>>> that are exposed via public APIs.
>>> I added serialVersionUID to a bunch of public java lang/net/util
>>> classes a wile back. I used jdk/bin/serialver to generate the
>>> serialVersionUID and verified that it was the same for previous
>>> versions of the class back to when the class was originally added. Not
>>> such a problem when you have archived jdk's back to 1.1.8 in /java/re
>>> in the US domain! Maybe Sasha could do something similar.
>> I used jdk/bin/serialver from JDK 7. Should I check if it's the same for
>> when the classes were originally added?
> Yes, if it is the same as when the class was originally added to the JDK that
> would most likely mean it has not changed since and it is safe to use.
>
> --Sean
>
>>> -Chris.
>>>
>>>> 2) Consider adding a comment explaining the
>>>> "@SuppressWarnings("unchecked")"
>>>> annotations
>> That sounds good. I'll post another webrev once I've added the comments.
>>>> --Sean
>>>>
>>>> On 7/12/11 4:16 PM, Alexandre Boulgakov wrote:
>>>>> Since yesterday's webrev, I've changed some unchecked casts to use
>>>>> Class<T>.cast(Object) instead, per Dave's suggestion.
>>>>> Updated webrev: http://cr.openjdk.java.net/~jjg/7064075.1/
>>>>>
>>>>> Also, the original webrev was posted under the wrong bug ID, so it's
>>>>> been moved to http://cr.openjdk.java.net/~jjg/7064075/.
>>>>>
>>>>> Thanks,
>>>>> Sasha
>>>>>
>>>>> On 7/12/2011 1:03 PM, Brad Wetmore wrote:
>>>>>> Sean/Valerie/Max/Xuelei,
>>>>>>
>>>>>>> Hello Brad,
>>>>>>>
>>>>>>> Could you please review these changes?
>>>>>> I'm swamped again, can one of you take a look at Sasha's changes?
>>>>>>
>>>>>> Brad
>>>>>>
>>>>>>
>>>>>>
>>>>>> On 7/11/2011 1:56 PM, Alexandre Boulgakov wrote:
>>>>>>> Hello Brad,
>>>>>>>
>>>>>>> Could you please review these changes?
>>>>>>>
>>>>>>> Bug detail:
>>>>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7064075
>>>>>>> webrev: http://cr.openjdk.java.net/~jjg/7076075/
>>>>>>>
>>>>>>> Summary:
>>>>>>>
>>>>>>> * Small changes to Java files to remove most build warnings.
>>>>>>> * Small changes to relevant makefiles to prevent
>>>>>>> reintroduction of
>>>>>>> removed warnings.
>>>>>>>
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Sasha Boulgakov
>>>>>>>
More information about the security-dev
mailing list