Re: RFR 8161474: Extract interface from java.net.http.RawChannel

2016-07-18 Thread Michael McMahon
Looks fine Pavel. - Michael On 18/07/2016, 11:29, Pavel Rappo wrote: Hello, Could you please review the following change for [1]? http://cr.openjdk.java.net/~prappo/8161474/webrev.00/ This change is purely a refactoring. No functionality changes are intended. RawChannel is an I/O

RFR 8161474: Extract interface from java.net.http.RawChannel

2016-07-18 Thread Pavel Rappo
Hello, Could you please review the following change for [1]? http://cr.openjdk.java.net/~prappo/8161474/webrev.00/ This change is purely a refactoring. No functionality changes are intended. RawChannel is an I/O abstraction that represents a WebSocket implementation's view on the

Re: RFR [9] 8160993: Fix headers in the java/net/http package

2016-07-18 Thread Alan Bateman
On 18/07/2016 11:26, Chris Hegarty wrote: This is a change fix the license headers in the java/net/http package. There was a cut'n'paste error that mostly missed the last line of the header. Also, I added a new line after the header consistently in every file. As well as a few places that had

RFR [9] 8160993: Fix headers in the java/net/http package

2016-07-18 Thread Chris Hegarty
This is a change fix the license headers in the java/net/http package. There was a cut'n'paste error that mostly missed the last line of the header. Also, I added a new line after the header consistently in every file. As well as a few places that had other minor issues.