You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by gn...@apache.org on 2008/10/23 20:41:12 UTC

svn commit: r707439 - /geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java

Author: gnodet
Date: Thu Oct 23 11:41:11 2008
New Revision: 707439

URL: http://svn.apache.org/viewvc?rev=707439&view=rev
Log:
SMX4-123: Fix possible NPE on AIX

Modified:
    geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java

Modified: geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java?rev=707439&r1=707438&r2=707439&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java (original)
+++ geronimo/gshell/trunk/gshell-support/gshell-clp/src/main/java/org/apache/geronimo/gshell/clp/Descriptor.java Thu Oct 23 11:41:11 2008
@@ -61,7 +61,8 @@
         }
         
         this.required = required;
-        this.handler = handler;
+        // On IBM JDK, the value passed is null instead of the default value, so fix it in case
+        this.handler = handler != null ? handler : Handler.class;
         this.multiValued = multiValued;
     }