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 <[email protected]> wrote: > This will cause warnings in client code, so better revert it? > > Am 11.06.2013 um 21:47 schrieb [email protected]: > >> 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: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
