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

Reply via email to