Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of simone.tripodi@gmail.com
 designates 209.85.214.175 as permitted sender)
MIME-Version: 1.0
Sender: simone.tripodi@gmail.com
In-Reply-To: <286AD3C4-9D4D-4C4D-8292-7E27FD4BF43B@gmail.com>
References: <20130611194716.86FE5238889B@eris.apache.org>
	<286AD3C4-9D4D-4C4D-8292-7E27FD4BF43B@gmail.com>
Date: Tue, 11 Jun 2013 22:00:04 +0200
Message-ID: 
 <CAAqLGLNKDQgOWgR5kb5Ztbb7rv+zn3=GwSk5gXguYfJrK_mcgg@mail.gmail.com>
Subject: Re: svn commit: r1491924 -
 /commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java
From: Simone Tripodi <simonetripodi@apache.org>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=UTF-8

I usually don't take too much care to this kind of warnings...

My opinion is you can feel free to keep the best option you like :P

best and thanks,
-Simo

http://people.apache.org/~simonetripodi/
http://twitter.com/simonetripodi


On Tue, Jun 11, 2013 at 9:50 PM, Benedikt Ritter <beneritter@gmail.com> wrote:
> This will cause warnings in client code, so better revert it?
>
> Am 11.06.2013 um 21:47 schrieb britter@apache.org:
>
>> Author: britter
>> Date: Tue Jun 11 19:47:16 2013
>> New Revision: 1491924
>>
>> URL: http://svn.apache.org/r1491924
>> Log:
>> Use var args instead of array as parameter
>>
>> Modified:
>>    commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java
>>
>> Modified: commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java
>> URL: http://svn.apache.org/viewvc/commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java?rev=1491924&r1=1491923&r2=1491924&view=diff
>> ==============================================================================
>> --- commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java (original)
>> +++ commons/proper/chain/trunk/core/src/main/java/org/apache/commons/chain2/impl/ChainBase.java Tue Jun 11 19:47:16 2013
>> @@ -69,7 +69,7 @@ public class ChainBase<K, V, C extends M
>>      *  or one of the individual {@link Command} elements,
>>      *  is <code>null</code>
>>      */
>> -    public ChainBase(Command<K, V, C>[] commands) {
>> +    public ChainBase(Command<K, V, C>... commands) {
>>         if (commands == null) {
>>             throw new IllegalArgumentException();
>>         }
>>
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org