Looks fine.
Thanks, Artem On 7/12/2013 6:36 PM, Sergey Bylokhov wrote:
Hello, Please review the fix for jdk 7/8. The problem is in incorrect merge between 8009012 and 8003559. I remove part(optimization) of the 8009012, because it was merged incorrectly and because it require additional testing after cpu merge. Also a call to flushBuffers was removed, when the peer is null, because it does not make sense in this case and can hang. Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8020298 Webrevs can be found at: http://cr.openjdk.java.net/~serb/8020298/webrev.8/ http://cr.openjdk.java.net/~serb/8020298/webrev.7/ Webrevs for jdk8 and jdk7 are different but result is the same.
