Regex Point Lambdafication Patch
forax at univ-mlv.fr
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 univ-mlv.fr> 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?
>> if the stream (or the spliterator in the middle) calls iterator.next()
>> 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() ?
>> 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'.
More information about the lambda-dev