make executor singleton again

1 files changed, 5 insertions, 7 deletions

diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/util/ExecutorServiceProvider.java b/app/src/main/java/it/niedermann/nextcloud/deck/util/ExecutorServiceProvider.java
index 93575f401..dc5e30bac 100644
--- a/app/src/main/java/it/niedermann/nextcloud/deck/util/ExecutorServiceProvider.java
+++ b/app/src/main/java/it/niedermann/nextcloud/deck/util/ExecutorServiceProvider.java
@@ -2,22 +2,20 @@ package it.niedermann.nextcloud.deck.util;
 
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
-import java.util.concurrent.LinkedBlockingQueue;
-import java.util.concurrent.ThreadPoolExecutor;
-import java.util.concurrent.TimeUnit;
 
 /**
  * If we really want <strong>this</strong>, we should default to {@link Executors#newWorkStealingPool()}.
  * Though I recommend to distinguish between blocking threads and non-blocking threads (like network operations), where it does not make sense to limit it to available CPU cores.
  */
-@Deprecated(forRemoval = true)
+//@Deprecated(forRemoval = true)
 public class ExecutorServiceProvider {
 
-    private static final int NUMBER_OF_CORES = Runtime.getRuntime().availableProcessors();
+//    private static final int NUMBER_OF_CORES = Runtime.getRuntime().availableProcessors();
 
     private static final ExecutorService EXECUTOR =
-            new ThreadPoolExecutor(NUMBER_OF_CORES>>1, NUMBER_OF_CORES,
-                60L, TimeUnit.SECONDS, new LinkedBlockingQueue<>());
+            Executors.newWorkStealingPool();
+//            new ThreadPoolExecutor(NUMBER_OF_CORES>>1, NUMBER_OF_CORES,
+//                60L, TimeUnit.SECONDS, new LinkedBlockingQueue<>());
 
     private ExecutorServiceProvider() {
         // hide Constructor