[jbpm-commits] JBoss JBPM SVN: r6689 - jbpm3/branches/jbpm-3.2-soa/enterprise-jee5/src/main/java/org/jbpm/jms.

do-not-reply at jboss.org do-not-reply at jboss.org
Tue Sep 28 08:02:38 EDT 2010


Author: bradsdavis
Date: 2010-09-28 08:02:37 -0400 (Tue, 28 Sep 2010)
New Revision: 6689

Modified:
   jbpm3/branches/jbpm-3.2-soa/enterprise-jee5/src/main/java/org/jbpm/jms/JmsConnectorService.java
Log:
Cleaned up the connector by putting the close in the finally block.

Modified: jbpm3/branches/jbpm-3.2-soa/enterprise-jee5/src/main/java/org/jbpm/jms/JmsConnectorService.java
===================================================================
--- jbpm3/branches/jbpm-3.2-soa/enterprise-jee5/src/main/java/org/jbpm/jms/JmsConnectorService.java	2010-09-28 11:52:37 UTC (rev 6688)
+++ jbpm3/branches/jbpm-3.2-soa/enterprise-jee5/src/main/java/org/jbpm/jms/JmsConnectorService.java	2010-09-28 12:02:37 UTC (rev 6689)
@@ -70,30 +70,36 @@
   }
 
   void sendWithoutSaving(Job job) {
+	  MessageProducer messageProducer = null;
+	  Session session = null;
     try {
-      Session session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
-      MessageProducer messageProducer = session.createProducer(factory.getDestination());
+      session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
+      messageProducer = session.createProducer(factory.getDestination());
       Message message = session.createMessage();
       populateMessage(message, job);
       messageProducer.send(message);
-      
-      try {
-          messageProducer.close();
-        }
-        catch (JMSException e) {
-          log.warn("could not close message producer", e);
-        }
-
-        try {
-          session.close();
-        }
-        catch (JMSException e) {
-          log.warn("could not close jms session", e);
-        }
     }
     catch (JMSException e) {
       throw new JbpmException("could not send jms message", e);
     }
+    finally {
+      if(messageProducer!=null) {
+    		try {
+    			messageProducer.close();
+  			}
+    		catch (JMSException e) {
+    	    	  log.warn("could not close message producer", e);
+    	    }
+	  }
+      if(session!=null) {
+	      try {
+	            session.close();
+	      }
+	      catch (JMSException e) {
+	    	  log.warn("could not close jms session", e);
+	      }
+      }
+    }
   }
 
   protected void populateMessage(Message message, Job job) throws JMSException {



More information about the jbpm-commits mailing list