Re: RFR JDK 9: 8032502: java.time add @param tags to readObject

2014-01-22 Thread Joe Darcy
On 01/22/2014 01:39 PM, roger riggs wrote: Thanks for the reminder Joe, I updated the webrev with the 2nd, shorter version. Ship it :-) -Joe Roger On 1/22/2014 4:33 PM, Joe Darcy wrote: On 01/22/2014 01:26 PM, roger riggs wrote: Please review this javadoc improvement to add @param tags

Re: RFR JDK 9: 8032502: java.time add @param tags to readObject

2014-01-22 Thread roger riggs
Thanks for the reminder Joe, I updated the webrev with the 2nd, shorter version. Roger On 1/22/2014 4:33 PM, Joe Darcy wrote: On 01/22/2014 01:26 PM, roger riggs wrote: Please review this javadoc improvement to add @param tags to readObject Webrev: http://cr.openjdk.java.net/~rriggs/web

Re: RFR JDK 9: 8032502: java.time add @param tags to readObject

2014-01-22 Thread Joe Darcy
On 01/22/2014 01:26 PM, roger riggs wrote: Please review this javadoc improvement to add @param tags to readObject Webrev: http://cr.openjdk.java.net/~rriggs/webrev-time-param-8032502/ Thanks, Roger Hi Roger, I'll approve this if you either * replace "ObjectInputStream" with "{@code Ob

Re: RFR JDK 9: 8032502: java.time add @param tags to readObject

2014-01-22 Thread Lance Andersen - Oracle
looks fine Roger as am sure this will make the doclint warnings less On Jan 22, 2014, at 4:26 PM, roger riggs wrote: > Please review this javadoc improvement to add @param tags to readObject > > Webrev: > > http://cr.openjdk.java.net/~rriggs/webrev-time-param-8032502/ > > Thanks, Roger >

RFR JDK 9: 8032502: java.time add @param tags to readObject

2014-01-22 Thread roger riggs
Please review this javadoc improvement to add @param tags to readObject Webrev: http://cr.openjdk.java.net/~rriggs/webrev-time-param-8032502/ Thanks, Roger