You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ui...@apache.org on 2013/10/07 15:04:15 UTC
svn commit: r1529831 -
/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java
Author: uiterlix
Date: Mon Oct 7 13:04:15 2013
New Revision: 1529831
URL: http://svn.apache.org/r1529831
Log:
Small performance improvement in SerialExecutor trough preventing exceptions.
Modified:
felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java
Modified: felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java?rev=1529831&r1=1529830&r2=1529831&view=diff
==============================================================================
--- felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java (original)
+++ felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/SerialExecutor.java Mon Oct 7 13:04:15 2013
@@ -77,12 +77,11 @@ public final class SerialExecutor {
private void scheduleNext() {
Runnable active;
synchronized (this) {
- try {
- m_active = (Runnable) m_workQueue.removeFirst();
- }
- catch (NoSuchElementException e) {
- m_active = null;
- }
+ if (!m_workQueue.isEmpty()) {
+ m_active = (Runnable) m_workQueue.removeFirst();
+ } else {
+ m_active = null;
+ }
active = m_active;
}
if (active != null) {