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

Posted to scm@geronimo.apache.org by dj...@apache.org on 2010/01/18 22:03:04 UTC

svn commit: r900557 - /geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java

Author: djencks
Date: Mon Jan 18 21:03:04 2010
New Revision: 900557

URL: http://svn.apache.org/viewvc?rev=900557&view=rev
Log:
add error checking for WrapperNamedXAResource constructor

Modified:
    geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java

Modified: geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java
URL: http://svn.apache.org/viewvc/geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java?rev=900557&r1=900556&r2=900557&view=diff
==============================================================================
--- geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java (original)
+++ geronimo/components/txmanager/trunk/geronimo-transaction/src/main/java/org/apache/geronimo/transaction/manager/WrapperNamedXAResource.java Mon Jan 18 21:03:04 2010
@@ -35,6 +35,8 @@
     private final String name;
 
     public WrapperNamedXAResource(XAResource xaResource, String name) {
+        if (xaResource == null) throw new NullPointerException("No XAResource supplied.  XA support may not be configured properly");
+        if (name == null) throw new NullPointerException("No name supplied. Resource adapter not properly configured");
         this.xaResource = xaResource;
         this.name = name;
     }