wbrawner/thunderbird-android
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Implemented TODO in CoreService

Browse source 
The todo:
    // TODO: remove this. we never set threadPool to null

    The fix:
    Removed most code that would be executed if threadPool were null
    Reordered a little logic to make code easier to read
This commit is contained in:
Joshua Nelson 2017-10-30 15:10:39 -04:00
parent d696723023
commit 4178306273
1 changed files with 10 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
k9mail/src/main/java/com/fsck/k9/service/CoreService.java
View file

@ -324,16 +324,6 @@ public abstract class CoreService extends Service {
} }
}; };
// TODO: remove this. we never set mThreadPool to null
if (mThreadPool == null) {
Timber.e("CoreService.execute (%s) called with no thread pool available; " +
"running Runnable %d in calling thread", className, runner.hashCode());
synchronized (this) {
myRunner.run();
serviceShutdownScheduled = startId != null;
}
} else {
Timber.d("CoreService (%s) queueing Runnable %d with startId %d", className, runner.hashCode(), startId); Timber.d("CoreService (%s) queueing Runnable %d with startId %d", className, runner.hashCode(), startId);
try { try {
@ -342,12 +332,11 @@ public abstract class CoreService extends Service {
} catch (RejectedExecutionException e) { } catch (RejectedExecutionException e) {
// Ignore RejectedExecutionException after we shut down the thread pool in // Ignore RejectedExecutionException after we shut down the thread pool in
// onDestroy(). Still, this should not happen! // onDestroy(). Still, this should not happen!
if (!mShutdown) { if (mShutdown) {
throw e;
}
Timber.i("CoreService: %s is shutting down, ignoring rejected execution exception: %s", Timber.i("CoreService: %s is shutting down, ignoring rejected execution exception: %s",
className, e.getMessage()); className, e.getMessage());
} else {
throw e;
} }
} }