Re: svn commit: r1449456 - in /commons/proper/beanutils/trunk/src: changes/changes.xml main/java/org/apache/commons/beanutils/ConstructorUtils.java test/java/org/apache/commons/beanutils/ConstructorUtilsTestCase.java

[Simone Tripodi]

Hi there Bene,

> @@ -82,6 +82,9 @@ public class ConstructorUtils {
>              InstantiationException {
>
>          Object[] args = { arg };
> +        if (arg == null) {
> +            args = null;
> +        }
>          return invokeConstructor(klass, args);
>      }

I'd invert the logic to

        Object[] args = null;
        if (arg != null) {
            args = { arg };
        }
        return invokeConstructor(klass, args);

HTH,
-Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/

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