Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-14 Thread Joseph D. Darcy
Hello, Not providing a code review, but I wanted to state I think this is a very good sort of cleanup and worthwhile to get into the platform once the details have been worked through. Thanks, -Joe On 5/12/2015 3:10 PM, Patrick Reinhart wrote: Hi Pavel, Is there anything I can do in the m

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-13 Thread Pavel Rappo
If you have time it might be useful to have a look for places to retrofit with java.nio.file.Files This class provides lots of useful tools which cover many usecases. Especially take a look at `copy` methods. Thanks! -Pavel > On 12 May 2015, at 23:10, Patrick Reinhart wrote: > > Hi Pavel, > >

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-13 Thread Chris Hegarty
On 13/05/15 12:24, patr...@reini.net wrote: Hi Chris, That would simplify the process greatly. What I exactly need to do for getting the JDK 9 project author role? See the section named "Becoming an Author" on the Projects page [1]. -Chris. [1] http://openjdk.java.net/projects/ Cheers, P

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-13 Thread patrick
Hi Chris, That would simplify the process greatly. What I exactly need to do for getting the JDK 9 project author role? Cheers, Patrick On 2015-05-13 12:12, Chris Hegarty wrote: Given your previous contributions [1] and ongoing involvement in OpenJDK, it would be reasonable for you to requ

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-13 Thread Chris Hegarty
On 12/05/15 23:10, Patrick Reinhart wrote: ... Unfortunately I do not have no account on cr.openjdk.java.net , so I need to paste the patch to the mailing list directly. Given your previous contributions [1] and ongoing involvement in OpenJDK, it would be reasona

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-12 Thread Patrick Reinhart
Hi Pavel, Is there anything I can do in the meantime? Unfortunately I do not have no account on cr.openjdk.java.net , so I need to paste the patch to the mailing list directly. Cheers Patrick > Am 10.05.2015 um 21:16 schrieb Chris Hegarty : > > I have not looked

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-10 Thread Patrick Reinhart
Hi Chris, > Am 10.05.2015 um 21:16 schrieb Chris Hegarty : > > I have not looked at the changes in detail yet, but I think it is going in > the right direction. Just a few comments: > >> On 9 May 2015, at 00:00, Patrick Reinhart > > wrote: >> >> diff -r 7101bcceb43d ma

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-10 Thread Chris Hegarty
I have not looked at the changes in detail yet, but I think it is going in the right direction. Just a few comments: > On 9 May 2015, at 00:00, Patrick Reinhart wrote: > > diff -r 7101bcceb43d make/src/classes/build/tools/module/ModuleArchive.java > --- a/make/src/classes/build/tools/module/Mod

Re: Updating existing JDK code to use InputStream.transferTo() (jdk)

2015-05-08 Thread Patrick Reinhart
Here the first batch of changes Cheers Patrick > Am 07.05.2015 um 20:44 schrieb Pavel Rappo : > > Hi Patrick, > > That's a good idea! I can sponsor them no problem. > > -Pavel diff -r 7101bcceb43d make/src/classes/build/tools/module/ModuleArchive.java --- a/make/src/classes/build/tools/modul