Thanks Jon for the review.
updated webrev : http://cr.openjdk.java.net/~pmuthuswamy/8203792/webrev.01/
I have removed the date from the comparison.
Regards,
Priya
On 8/14/2018 5:43 AM, Jonathan Gibbons wrote:
On 8/2/18 4:31 AM, Priya Lakshmi Muthuswamy wrote:
Hi,
Kindly review the fix for
https://bugs.openjdk.java.net/browse/JDK-8203792
webrev: http://cr.openjdk.java.net/~pmuthuswamy/8203792/webrev.00/
Thanks,
Priya
In the new test, the use of LocalDate.now() will lead to occasional
spurious test
failures, when now() isn't the same as it was when javadoc wrote the
file.
I don't have a good suggestion, other than to exclude the exact date
from the
comparison. Anything more sophisticated would involve reading the file to
find that line, and find the date on the line, and verify the value looks
"reasonable". In between, there would be the ability to do regex matching
on the file contents, but we didn't get around to designing/supporting
that yet.
-- Jon
