Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-23 Thread Patrick Reinhart
On 19.08.2016 00:17, Paul Sandoz wrote: > > Onward to the implementation! > > Paul. I did not manage it to get the OpenJDK to compile stressfully under my Fedora Linux as in the past now. Anyhow here a sample implementation, that I will integrate into the existing ClassLoader and the according T

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-18 Thread Paul Sandoz
> On 17 Aug 2016, at 02:39, Patrick Reinhart wrote: > > Hi Paul, > > Thanks for the input. I integrated that into the version here: > > http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.03 > Onward to the implementation! Paul.

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-17 Thread Patrick Reinhart
On 17.08.2016 01:50, Paul Sandoz wrote: > On 8 Aug 2016, at 12:14, Patrick Reinhart wrote: >> Am 08.08.2016 um 18:55 schrieb Alan Bateman : >>> On 08/08/2016 17:29, Patrick Reinhart wrote: >>> : I tried to integrate your suggested changes here: http://cr.openjdk.java.net/~reinhapa/r

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-16 Thread Paul Sandoz
> On 8 Aug 2016, at 12:14, Patrick Reinhart wrote: > > >> Am 08.08.2016 um 18:55 schrieb Alan Bateman : >> >> On 08/08/2016 17:29, Patrick Reinhart wrote: >> >>> : >>> I tried to integrate your suggested changes here: >>> http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_Stream

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-08 Thread Patrick Reinhart
> Am 08.08.2016 um 18:55 schrieb Alan Bateman : > > On 08/08/2016 17:29, Patrick Reinhart wrote: > >> : >> I tried to integrate your suggested changes here: >> http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.01 >> > I should have been clearer. A lazy implementatio

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-08 Thread Alan Bateman
On 08/08/2016 17:29, Patrick Reinhart wrote: : I tried to integrate your suggested changes here: http://cr.openjdk.java.net/~reinhapa/reviews/8161230/ClassLoader_StreamMethods.01 I should have been clearer. A lazy implementation of resources/systemResources methods won't throw any exceptions,

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-08 Thread Patrick Reinhart
On 05.08.2016 06:18, Alan Bateman wrote: > On 04/08/2016 10:33, Patrick Reinhart wrote: > >> I was quit busy lately and this comes a bit late, I guess you do not >> have less work ;-) >> >> On 15.07.2016 17:10, Paul Sandoz wrote: When I understand you correctly here we should concentrate on th

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-05 Thread Gregg Wonderly
Sent from my iPad > On Aug 4, 2016, at 11:18 PM, Alan Bateman wrote: > >> On 04/08/2016 10:33, Patrick Reinhart wrote: >> >> Hi Paul, >> >> I was quit busy lately and this comes a bit late, I guess you do not have >> less work ;-) >> >> On 15.07.2016 17:10, Paul Sandoz wrote: When I u

Re: RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-04 Thread Alan Bateman
On 04/08/2016 10:33, Patrick Reinhart wrote: Hi Paul, I was quit busy lately and this comes a bit late, I guess you do not have less work ;-) On 15.07.2016 17:10, Paul Sandoz wrote: When I understand you correctly here we should concentrate on the public methods naming firstly? I initially w

RFR: JDK-8161230 - Create java.util.stream.Stream from Iterator / Enumeration

2016-08-04 Thread Patrick Reinhart
Hi Paul, I was quit busy lately and this comes a bit late, I guess you do not have less work ;-) On 15.07.2016 17:10, Paul Sandoz wrote: >> When I understand you correctly here we should concentrate on the public >> methods naming firstly? I initially was not sure, what a proper naming >> for th