svn commit: r1818547 - /ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1818547 - /ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java

mbrohl
Author: mbrohl
Date: Mon Dec 18 10:11:40 2017
New Revision: 1818547

URL: http://svn.apache.org/viewvc?rev=1818547&view=rev
Log:
Improved: General refactoring and code improvements, package
org.apache.ofbiz.base.concurrent.
(OFBIZ-9928)

Thanks Dennis Balkir for reporting and providing the patches.

Modified:
    ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java

Modified: ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java?rev=1818547&r1=1818546&r2=1818547&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/concurrent/ExecutionPool.java Mon Dec 18 10:11:40 2017
@@ -81,9 +81,7 @@ public final class ExecutionPool {
         for (Future<F> future: futureList) {
             try {
                 result.add(future.get());
-            } catch (ExecutionException e) {
-                Debug.logError(e, module);
-            } catch (InterruptedException e) {
+            } catch (ExecutionException | InterruptedException e) {
                 Debug.logError(e, module);
             }
         }
@@ -149,7 +147,9 @@ public final class ExecutionPool {
                 return 0;
             }
             long r = timeDiff((Pulse) other);
-            if (r < 0) return -1;
+            if (r < 0) {
+                return -1;
+            }
             return 1;
         }