On Tue, Aug 23, 2011 at 10:41 AM, Jörn Kottmann <[email protected]> wrote:

> On 8/23/11 3:40 PM, [email protected] wrote:
>
>> On Tue, Aug 23, 2011 at 5:31 AM, Jörn Kottmann<[email protected]>
>>  wrote:
>>
>>  On 8/23/11 6:07 AM, [email protected] wrote:
>>>
>>>  I added variable length param only for the Chunker. Please check if it
>>>> looks
>>>> OK. If yes I will do it for the other tools.
>>>>
>>>>  Looks good to me, you don't need the null check for the variable length
>>> param. If the user passes null, he gets a NullPointerException.
>>>
>>> In the case he doesn't pass anything you get an empty array.
>>>
>>> What do you think, is the variable length param way better than the
>>> other?
>>>
>>>  Yes, it looks nicer. I will do it for the other tools now.
>>
>>  +1
>
> Jörn
>

I finished it. If you agree with the changes I will close the issue.

Reply via email to