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]