On 14 Jan 2014, at 10:53, Chris Hegarty <chris.hega...@oracle.com> wrote:
> On 14 Jan 2014, at 10:51, Alan Bateman <alan.bate...@oracle.com> wrote: > >> On 14/01/2014 08:28, Chris Hegarty wrote: >>> On 13 Jan 2014, at 19:29, Chris Hegarty<chris.hega...@oracle.com> wrote: >>> >>>> On 13 Jan 2014, at 19:23, Dimitar Mavrodiev<dmavrod...@gmail.com> wrote: >>>> >>>>> Hi Chris, >>>>> >>>>> I can't open the link, but yet I wouldn't mind if you folded your patch >>>>> into mine. >>>> How embarrassing. Our public facing server is temporarily offline so I >>>> copied the webrev to a temporary server just to verify it, then obviously >>>> forgot it was an internal link. Sorry about this. I’ll repost a mail when >>>> the public server is back online. >>> http://cr.openjdk.java.net/~chegar/7100957/webrev.00/webrev/ >>> >> The Buffered*, flush and try-with-resource changes look good. So will there >> be a combined webrev with everything? Also about about Dimi's change to >> readSocksReply to handle fragmentation, should that be included too? > > Yes, it is in my patch queue. I just created the diffs on top of that. I will > fold the patches into one and can re-post the updated webrev. For the archives: http://cr.openjdk.java.net/~chegar/7100957/webrev.01/ -Chris. > > -Chris. > >> >> -Alan. >
