Daniel, On 06/04/17 11:32, Daniel Fuchs wrote:
...webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8170940/webrev.00/
Looks good Daniel. Just a few comments. 1) Http1Exchange.java Can 'operations' now be made private, and not a synchronizedList? Now that it is operated on only within synchronized blocks. 2) Exchange.java L197 : synchronized(this) I'm not sure what this synchronized block gets you? The code is racy, and both fields are volatile. Can this be removed? -Chris.
