In TestNavigation.java, line 147, there i9s no need to say "for HTML5"
Fix that and you're OK.
-- Jon
On 03/05/2019 10:55 PM, Priya Lakshmi Muthuswamy wrote:
Hi Jon,
I have removed the HtmlVersion as well.
updated webrev :
http://cr.openjdk.java.net/~pmuthuswamy/8220087/webrev.01/
Thanks,
Priya
On 3/6/2019 5:51 AM, Jonathan Gibbons wrote:
Priya,
I'd be inclined to remove HtmlVersion as well. That would include
the parameter to WorkArounds.initDocLint, replacing the usage on line
140 with a plain lower-case string.
-- Jon
On 03/05/2019 12:54 AM, Priya Lakshmi Muthuswamy wrote:
Hi,
Kindly review the changes for
https://bugs.openjdk.java.net/browse/JDK-8220087
webrev : http://cr.openjdk.java.net/~pmuthuswamy/8220087/webrev.00/
Test
test/langtools/jdk/javadoc/doclet/testNavigation/TestNavigation.java
had duplicate tests, removed one of them.
Thanks,
Priya