commit 6595927d3a19438e971d0fcb3546e7477ea452b4
author George G <kreach3r@users.noreply.github.com> Mon Mar 06 21:24:20 2017 +0200
committer Nathan Chancellor <natechancellor@gmail.com> Tue Mar 07 01:58:11 2017 -0700
tree d1ea566535f7bbc6f9644e43f1a97963c2c27ea6
parent e9e6979ecd441be0766735c029f84c798429e7f1

Run RestartService immediately if called

There is no reason to add the restartservice command to the job list.
If we are going to forcibly restart the service, it means that
something bad happened and the job list itself may be unresponsive.

Change-Id: I171af32081cf92d58616cdecc0413b8c1443dd27

app/src/main/java/projekt/interfacer/services/JobService.java

diff --git a/app/src/main/java/projekt/interfacer/services/JobService.java b/app/src/main/java/projekt/interfacer/services/JobService.java
index 9e896bb..d352a16 100644
--- a/app/src/main/java/projekt/interfacer/services/JobService.java
+++ b/app/src/main/java/projekt/interfacer/services/JobService.java
@@ -220,7 +220,8 @@
         } else if (TextUtils.equals(command, COMMAND_VALUE_RESTART_UI)) {
             jobs_to_add.add(new UiResetJob());
         } else if (TextUtils.equals(command, COMMAND_VALUE_RESTART_SERVICE)) {
-            jobs_to_add.add(new RestartServiceJob());
+            log("Restarting JobService...");
+            restartService();
         } else if (TextUtils.equals(command, COMMAND_VALUE_CONFIGURATION_SHIM)) {
             jobs_to_add.add(new LocaleChanger(getApplicationContext(), mMainHandler));
         } else if (TextUtils.equals(command, COMMAND_VALUE_BOOTANIMATION)) {
@@ -841,17 +842,6 @@
         }
     }
 
-    private class RestartServiceJob implements Runnable {
-        @Override
-        public void run() {
-            log("Restarting JobService...");
-            Message message = mJobHandler.obtainMessage(JobHandler.MESSAGE_DEQUEUE,
-                    RestartServiceJob.this);
-            mJobHandler.sendMessage(message);
-            restartService();
-        }
-    }
-
     private class FontsJob implements Runnable {
         boolean mClear;
         String mPid;