Don't throw is stopping threads twice

This crash has been seen. Seems OK to be a bit kinder regarding
stopThreads(). Not touching startThreads() for now.

1 files changed, 6 insertions, 1 deletions

diff --git a/src/main/java/se/lublin/humla/net/HumlaNetworkThread.java b/src/main/java/se/lublin/humla/net/HumlaNetworkThread.java
index 028f4fc..68ecfed 100644
--- a/src/main/java/se/lublin/humla/net/HumlaNetworkThread.java
+++ b/src/main/java/se/lublin/humla/net/HumlaNetworkThread.java
@@ -19,10 +19,13 @@ package se.lublin.humla.net;
 
 import android.os.Handler;
 import android.os.Looper;
+import android.util.Log;
 
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 
+import se.lublin.humla.Constants;
+
 /**
  * Base class for TCP/UDP protocol implementations.
  * Provides a common threading model (single threaded queue for write)
@@ -54,7 +57,9 @@ public abstract class HumlaNetworkThread implements Runnable {
 
     protected void stopThreads() {
         if (!mInitialized) {
-            throw new IllegalArgumentException("Threads already shutdown.");
+            // TODO? Used throw, like startThreads()
+            Log.e(Constants.TAG, "Error in stopThreads: Threads already shutdown");
+            return;
         }
         mSendExecutor.shutdown();
         mReceiveExecutor.shutdownNow();