[jboss-cvs] JBoss Messaging SVN: r1999 - trunk/src/main/org/jboss/jms/server/endpoint.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Sat Jan 20 03:51:32 EST 2007
Author: ovidiu.feodorov at jboss.com
Date: 2007-01-20 03:51:32 -0500 (Sat, 20 Jan 2007)
New Revision: 1999
Modified:
trunk/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java
Log:
minor reformatting
Modified: trunk/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java
===================================================================
--- trunk/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java 2007-01-19 20:19:05 UTC (rev 1998)
+++ trunk/src/main/org/jboss/jms/server/endpoint/ServerConsumerEndpoint.java 2007-01-20 08:51:32 UTC (rev 1999)
@@ -218,21 +218,22 @@
return delivery;
}
- long deliveryId = sessionEndpoint.addDelivery(delivery, id, dlq, expiryQueue, redeliveryDelay);
+ long deliveryId =
+ sessionEndpoint.addDelivery(delivery, id, dlq, expiryQueue, redeliveryDelay);
// We don't send the message as-is, instead we create a MessageProxy instance. This allows
// local fields such as deliveryCount to be handled by the proxy but global data to be
// fielded by the same underlying Message instance. This allows us to avoid expensive
// copying of messages
- MessageProxy mp = JBossMessage.createThinDelegate(deliveryId, message, ref.getDeliveryCount());
+ MessageProxy mp = JBossMessage.
+ createThinDelegate(deliveryId, message, ref.getDeliveryCount());
- //We send the message to the client on the current thread.
- //The message is written onto the transport and then the thread returns immediately
- //without waiting for a response
+ // We send the message to the client on the current thread. The message is written onto the
+ // transport and then the thread returns immediately without waiting for a response.
- //FIXME - how can we ensure that a later send doesn't overtake an earlier send - this might
- //happen if they are using different underlying TCP connections (e.g. from pool)
+ // FIXME - how can we ensure that a later send doesn't overtake an earlier send - this
+ // might happen if they are using different underlying TCP connections (e.g. from pool)
ClientDelivery del = new ClientDelivery(mp, id);
@@ -242,10 +243,9 @@
try
{
- //FIXME - when the latest remoting changes make it into a release, we need
- //to use server side one way callbacks here.
- //I.e. we want to sent the invocation to the transport on the this thread and
- //return immediately without waiting for a response.
+ // FIXME - when the latest remoting changes make it into a release, we need to use
+ // server side one way callbacks here. I.e. we want to sent the invocation to the
+ // transport on the this thread and return immediately without waiting for a response.
callbackHandler.handleCallback(callback);
}
catch (HandleCallbackException e)
More information about the jboss-cvs-commits
mailing list