From b7f5bfe55e8a665bb94400a6cf230a0e5f837758 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sat, 21 Nov 2020 10:50:23 +0100
Subject: [PATCH] Revert "Prevent crash"

This reverts commit 9b0c4e572efcb9ae0bc83cda311d46d419ff2acd.
---
 .../java/eu/faircode/email/SimpleTask.java    | 20 +++++++------------
 1 file changed, 7 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/SimpleTask.java b/app/src/main/java/eu/faircode/email/SimpleTask.java
index 662646af6e..150a3761fd 100644
--- a/app/src/main/java/eu/faircode/email/SimpleTask.java
+++ b/app/src/main/java/eu/faircode/email/SimpleTask.java
@@ -130,7 +130,8 @@ public abstract class SimpleTask<T> implements LifecycleObserver {
         try {
             onPreExecute(args);
         } catch (Throwable ex) {
-            error(args, ex);
+            Log.e(ex);
+            onException(args, ex);
         }
 
         future = getExecutor(context).submit(new Runnable() {
@@ -194,7 +195,8 @@ public abstract class SimpleTask<T> implements LifecycleObserver {
                         try {
                             onPostExecute(args);
                         } catch (Throwable ex) {
-                            error(args, ex);
+                            Log.e(ex);
+                            onException(args, ex);
                         } finally {
                             try {
                                 if (ex == null) {
@@ -207,9 +209,10 @@ public abstract class SimpleTask<T> implements LifecycleObserver {
 
                                     onExecuted(args, (T) data);
                                 } else
-                                    error(args, ex);
+                                    onException(args, ex);
                             } catch (Throwable ex) {
-                                error(args, ex);
+                                Log.e(ex);
+                                onException(args, ex);
                             }
                         }
                     }
@@ -218,15 +221,6 @@ public abstract class SimpleTask<T> implements LifecycleObserver {
         });
     }
 
-    void error(Bundle args, Throwable ex) {
-        try {
-            Log.e(ex);
-            onException(args, ex);
-        } catch (Throwable exex) {
-            Log.e(exex);
-        }
-    }
-
     void cancel(Context context) {
         if (future != null)
             if (future.cancel(false)) {