Re: RFR: JDK-8167992 : Update documentation of java.util.Date class

2016-10-17 Thread Masayoshi Okutsu
+1 Masayoshi On 10/17/2016 8:05 PM, Yuka Kamiya wrote: Looks good to me. Thanks, -- Yuka On 2016/10/17 19:14, Rachna Goel wrote: Hi Masayoshi, Thanks for the review. I have updated href to have double quotes. http://cr.openjdk.java.net/~rgoel/JDK-8167992/webrev.01/ Thanks, Rachna On 10

Re: RFR: JDK-8167992 : Update documentation of java.util.Date class

2016-10-17 Thread Yuka Kamiya
Looks good to me. Thanks, -- Yuka On 2016/10/17 19:14, Rachna Goel wrote: Hi Masayoshi, Thanks for the review. I have updated href to have double quotes. http://cr.openjdk.java.net/~rgoel/JDK-8167992/webrev.01/ Thanks, Rachna On 10/17/16 3:00 PM, Masayoshi Okutsu wrote: Hi Rachna, Use of

Re: RFR: JDK-8167992 : Update documentation of java.util.Date class

2016-10-17 Thread Rachna Goel
Hi Masayoshi, Thanks for the review. I have updated href to have double quotes. http://cr.openjdk.java.net/~rgoel/JDK-8167992/webrev.01/ Thanks, Rachna On 10/17/16 3:00 PM, Masayoshi Okutsu wrote: Hi Rachna, Use of double quote is preferable for href. (i.e., href="http://...";>). Otherwise,

Re: RFR: JDK-8167992 : Update documentation of java.util.Date class

2016-10-17 Thread Masayoshi Okutsu
Hi Rachna, Use of double quote is preferable for href. (i.e., href="http://...";>). Otherwise, the fix looks good to me. Thanks, Masayoshi On 10/14/2016 6:46 PM, Rachna Goel wrote: Hi, Please review a simple documentation fix for java.util.Date class. Bug : https://bugs.openjdk.java.net/br

RFR: JDK-8167992 : Update documentation of java.util.Date class

2016-10-14 Thread Rachna Goel
Hi, Please review a simple documentation fix for java.util.Date class. Bug : https://bugs.openjdk.java.net/browse/JDK-8167992 webrev : http://cr.openjdk.java.net/~rgoel/JDK-8167992/webrev/ Fix : updated links pointing to old websites. Thanks, Rachna