Regex Point Lambdafication Patch

Remi Forax forax at
Wed Mar 13 07:43:45 PDT 2013

On 03/13/2013 01:19 PM, Paul Sandoz wrote:
> On Mar 13, 2013, at 12:09 AM, Remi Forax <forax at> wrote:
>> On 03/12/2013 10:33 PM, Ben Evans wrote:
>>> Thanks Brian.
>>> Does this webrev address the current comments?
>>> By the way, I've noticed that something in the webrev process appears
>>> to be mangling the Unicode chars in some of my tests. Is this OK? Is
>>> it known behaviour?
>>> Thanks,
>>> Ben
>> Ben,
>> if the stream (or the spliterator in the middle) calls
>> without calling iterator.hasNext(), your code doesn't work. Also because
>> null is not a possible value for lastChunk, you don't need a flag
>> (takenLastChunk) here.
> Annoyingly it is not possible to add a test to the stream test scenarios covering that case.

Is it because the Spliterator that wraps the Iterator always calls 
hasNext() before calling next() ?

>> Brian,
>> I wonder if having Spliterator.split() returning null as a default
>> implementation is a good idea.
>> It's far easier to write a Spliterator than an Iterator so either we
>> provide a method that takes a Spliterator that only works serially and
>> transform it to a parallel one or the default implementation of
>> Spliterator.split() should do something more meaningful than 'return null'.
> +1.
> Paul.


More information about the lambda-dev mailing list