Author: orudyy
Date: Thu Oct  6 16:18:51 2016
New Revision: 1763622

URL: http://svn.apache.org/viewvc?rev=1763622&view=rev
Log:
QPID-7447: [Java Broker] Avoid empty iterator creation on iteration through 
tickers in AggregateTicker

Modified:
    
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/transport/AggregateTicker.java

Modified: 
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/transport/AggregateTicker.java
URL: 
http://svn.apache.org/viewvc/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/transport/AggregateTicker.java?rev=1763622&r1=1763621&r2=1763622&view=diff
==============================================================================
--- 
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/transport/AggregateTicker.java
 (original)
+++ 
qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/transport/AggregateTicker.java
 Thu Oct  6 16:18:51 2016
@@ -32,9 +32,12 @@ public class AggregateTicker implements
     public int getTimeToNextTick(final long currentTime)
     {
         int nextTick = Integer.MAX_VALUE;
-        for(Ticker ticker : _tickers)
+        if (!_tickers.isEmpty())
         {
-            nextTick = Math.min(ticker.getTimeToNextTick(currentTime), 
nextTick);
+            for (Ticker ticker : _tickers)
+            {
+                nextTick = Math.min(ticker.getTimeToNextTick(currentTime), 
nextTick);
+            }
         }
         return nextTick;
     }
@@ -68,11 +71,14 @@ public class AggregateTicker implements
     @Override
     public void notifySchedulingDelay(final long schedulingDelay)
     {
-        for (Ticker ticker : _tickers)
+        if (!_tickers.isEmpty())
         {
-            if (ticker instanceof SchedulingDelayNotificationListener)
+            for (Ticker ticker : _tickers)
             {
-                ((SchedulingDelayNotificationListener) 
ticker).notifySchedulingDelay(schedulingDelay);
+                if (ticker instanceof SchedulingDelayNotificationListener)
+                {
+                    ((SchedulingDelayNotificationListener) 
ticker).notifySchedulingDelay(schedulingDelay);
+                }
             }
         }
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to