RFR: 8196027: Remove "Prev" and "Next" links from the javadoc navigation
Bhavesh Patel
bhavesh.x.patel at oracle.com
Thu Jan 25 05:23:34 UTC 2018
Please see my response inline. I have made the recommended changes and
have uploaded the updated webrev at
http://cr.openjdk.java.net/~bpatel/8196027/webrev.01/. There are a few
new files in this review from which I have deleted the dead code.
On 1/24/2018 4:44 PM, Jonathan Gibbons wrote:
>
>
> On 01/24/2018 01:25 PM, Bhavesh Patel wrote:
>> Hi,
>> Please review the fix for change in javadoc navigation bar.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8196027.
>>
>> Webrev: http://cr.openjdk.java.net/~bpatel/8196027/webrev.00/
>>
>> Regards,
>> Bhavesh.
>
>
> In SplitIndexWriter, there are still remnants remaining, such as
> prev/next fields.
> See lines 63-73.
>
> In Contents, there are more remnants, such as the set of prev/next
> labels.
> See lines 143-147, 158-162
Yes. Missed those unused variables. I have removed them from the classes
you mentioned and others where its no longer used. Also, I have removed
unused imports from these files.
>
> In the new test, checkOutput(file, false, ...) tests are notoriously
> weak and fragile.
> In general you want to make them as broad as possible. To that end, I
> would suggest
> turning most "+" in lines 81-111 into a comma at the end of the
> previous line,
> and reduce all the strings down to either "Next"/"Prev" or
> "Next WORD"/"Prev WORD".
I have updated the test.
>
> -- Jon
More information about the javadoc-dev
mailing list