RFR: CODETOOLS-7901044: Add next/prev links to navigate between diffs
David Chase
david.r.chase at oracle.com
Wed Oct 1 13:59:54 UTC 2014
I thought we had that in what I know of as “webrev-new”, see for example:
http://cr.openjdk.java.net/~drchase/8037816/webrev-9.05-09b/src/cpu/x86/vm/c1_LinearScan_x86.cpp.sdiff.html
But yours is also nice :-).
David
On 2014-10-01, at 4:56 AM, Bengt Rutisson <bengt.rutisson at oracle.com> wrote:
>
> 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