Code Review Request: 8025763

Anthony Scarpino anthony.scarpino at oracle.com
Wed Nov 27 20:46:06 UTC 2013


> On Nov 27, 2013, at 10:12 AM, Sean Mullan <sean.mullan at oracle.com> wrote:
> 
>> On 11/26/2013 08:20 PM, Bradford Wetmore wrote:
>> Tony,
>> 
>> I note the @since's are missing for the new methods, both in the
>> generated output in the overridden methods (i.e. no javadoc), and the
>> methods in which you've changed the behavior (i.e. new javadoc).
>> 
>> I'm not sure what you can do about the previous behavior (cc'ing
>> Mike/Sowmya, maybe they know), but what about the new ones?
> 
> That kind of seems like a javadoc bug to me. Shouldn't it add the @since tag as part of inheriting the javadoc?
> 
> Anyway, adding @since tags would be considered a docs only change, so Tony you could still fix this for JDK 8 - can you file a bug?
> 
> Thanks,
> Sean

My first thought was it would be a javadoc issue too.

Let me see if setting @since doesn't cause the inheritance of the other tag to stop. If it doesn't I'll fix it for 8. 

Tony

> 
>> 
>> Brad
>> 
>> 
>> 
>>> On 10/21/2013 2:48 PM, Sean Mullan wrote:
>>>> On 10/18/2013 10:52 PM, Anthony Scarpino wrote:
>>>> I've updated the webrev
>>>> 
>>>> http://cr.openjdk.java.net/~ascarpino/8025763/webrev.01/
>>> 
>>> Update looks good.
>>> 
>>> --Sean
> 



More information about the security-dev mailing list