RFR : 8203792 : Remove "compatibility" features from Head.java
Priya Lakshmi Muthuswamy
priya.lakshmi.muthuswamy at oracle.com
Tue Aug 14 04:37:38 UTC 2018
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
More information about the javadoc-dev
mailing list