Alan, Chris, it seems I had some problems uploading the webrev, I have reuploaded it to the same place -- http://cr.openjdk.java.net/~iignatyev//8181761/webrev.02/index.html
-- Igor > On Jun 9, 2017, at 2:01 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > > On 09/06/2017 07:42, Igor Ignatyev wrote: >> here is the refreshed version of patch : >> http://cr.openjdk.java.net/~iignatyev//8181761/webrev.02/index.html >> <http://cr.openjdk.java.net/%7Eiignatyev//8181761/webrev.02/index.html> >> >> there might be some new @build actions, b/c I have analyzed class files from >> all tests instead of looking only at ones which have jdk/test/lib classes. >> > You might need to scp the webrev again as the links to the diffs aren't > working. The #lines changed, as listed in index.html, looks promising but the > change-set is the previous change-set so I suspect something went wrong when > you published this version. > > -Alan > On Jun 9, 2017, at 3:46 AM, Chris Hegarty <chris.hega...@oracle.com> wrote: > I get 404 Not Found on many links, e.g. > http://cr.openjdk.java.net/~iignatyev//8181761/webrev.02/test/java/nio/channels/DatagramChannel/Promiscuous.java.sdiff.html