RFR: CODETOOLS-7901044: Add next/prev links to navigate between diffs

Bengt Rutisson bengt.rutisson at oracle.com
Wed Oct 1 08:56:41 UTC 2014


Hi everyone,

(This is a patch for webrev, but I am cross posting to the JDK 9 dev 
list since I'm not sure how many are on the webrev-dev list and most JDK 
developers might have opinions about this change.)

Can I get a couple of reviews for this small enhancement to the webrev tool?

CODETOOLS-7901044: Add next/prev links to navigate between diffs
https://bugs.openjdk.java.net/browse/CODETOOLS-7901044

Webrev:
http://cr.openjdk.java.net/~brutisso/CODETOOLS-7901044/webrev.00/


Background:
When browsing review requests that span many files it is quite time 
consuming to have to go back to the index page to view to the next or 
previous file. This patch adds links to the next and previous files in 
the diff views (including frames).

The above webrev is generated with this new version, but since it only 
contains a single file it is not that interesting. Here is an example of 
what the new webrev looks like for a larger change (the G1 class 
unloading change):

http://cr.openjdk.java.net/~brutisso/CODETOOLS-7901044/webrev.00/example/

Thanks,
Bengt



More information about the jdk9-dev mailing list