Code style improvements

4 files changed, 52 insertions, 49 deletions

diff --git a/core/src/main/java/de/danoeh/antennapod/core/service/PlayerWidgetJobService.java b/core/src/main/java/de/danoeh/antennapod/core/service/PlayerWidgetJobService.java
index 49f3058ef..2fd790ac7 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/service/PlayerWidgetJobService.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/service/PlayerWidgetJobService.java
@@ -31,9 +31,6 @@ public class PlayerWidgetJobService extends JobIntentService {
     private PlaybackService playbackService;
     private final Object waitForService = new Object();
 
-    public PlayerWidgetJobService() {
-    }
-
     public static void updateWidget(Context context) {
         enqueueWork(context, PlayerWidgetJobService.class, 0, new Intent(context, PlayerWidgetJobService.class));
     }
diff --git a/core/src/main/java/de/danoeh/antennapod/core/storage/DBTasks.java b/core/src/main/java/de/danoeh/antennapod/core/storage/DBTasks.java
index d1713ff99..02098e9ed 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/storage/DBTasks.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/storage/DBTasks.java
@@ -4,6 +4,7 @@ import android.content.Context;
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.database.Cursor;
+import android.support.annotation.Nullable;
 import android.support.v4.content.ContextCompat;
 import android.util.Log;
 
@@ -150,42 +151,43 @@ public final class DBTasks {
      *
      * @param context  Might be used for accessing the database
      * @param feeds    List of Feeds that should be refreshed.
-     * @param callback Called after everything was added enqueued for download
+     * @param callback Called after everything was added enqueued for download. Might be null.
      */
-    public static void refreshAllFeeds(final Context context, final List<Feed> feeds, Runnable callback) {
-        if (isRefreshing.compareAndSet(false, true)) {
-            new Thread(() -> {
-                if (feeds != null) {
-                    refreshFeeds(context, feeds);
-                } else {
-                    refreshFeeds(context, DBReader.getFeedList());
-                }
-                isRefreshing.set(false);
+    public static void refreshAllFeeds(final Context context, final List<Feed> feeds, @Nullable Runnable callback) {
+        if (!isRefreshing.compareAndSet(false, true)) {
+            Log.d(TAG, "Ignoring request to refresh all feeds: Refresh lock is locked");
+            return;
+        }
 
-                SharedPreferences prefs = context.getSharedPreferences(PREF_NAME, MODE_PRIVATE);
-                prefs.edit().putLong(PREF_LAST_REFRESH, System.currentTimeMillis()).apply();
+        new Thread(() -> {
+            if (feeds != null) {
+                refreshFeeds(context, feeds);
+            } else {
+                refreshFeeds(context, DBReader.getFeedList());
+            }
+            isRefreshing.set(false);
 
-                if (FlattrUtils.hasToken()) {
-                    Log.d(TAG, "Flattring all pending things.");
-                    new FlattrClickWorker(context).executeAsync(); // flattr pending things
+            SharedPreferences prefs = context.getSharedPreferences(PREF_NAME, MODE_PRIVATE);
+            prefs.edit().putLong(PREF_LAST_REFRESH, System.currentTimeMillis()).apply();
 
-                    Log.d(TAG, "Fetching flattr status.");
-                    new FlattrStatusFetcher(context).start();
+            if (FlattrUtils.hasToken()) {
+                Log.d(TAG, "Flattring all pending things.");
+                new FlattrClickWorker(context).executeAsync(); // flattr pending things
 
-                }
-                if (ClientConfig.gpodnetCallbacks.gpodnetEnabled()) {
-                    GpodnetSyncService.sendSyncIntent(context);
-                }
-                Log.d(TAG, "refreshAllFeeds autodownload");
-                autodownloadUndownloadedItems(context);
+                Log.d(TAG, "Fetching flattr status.");
+                new FlattrStatusFetcher(context).start();
 
-                if (callback != null) {
-                    callback.run();
-                }
-            }).start();
-        } else {
-            Log.d(TAG, "Ignoring request to refresh all feeds: Refresh lock is locked");
-        }
+            }
+            if (ClientConfig.gpodnetCallbacks.gpodnetEnabled()) {
+                GpodnetSyncService.sendSyncIntent(context);
+            }
+            Log.d(TAG, "refreshAllFeeds autodownload");
+            autodownloadUndownloadedItems(context);
+
+            if (callback != null) {
+                callback.run();
+            }
+        }).start();
     }
 
     /**
diff --git a/core/src/main/java/de/danoeh/antennapod/core/util/playback/Playable.java b/core/src/main/java/de/danoeh/antennapod/core/util/playback/Playable.java
index bb9896f25..ff7f5b79d 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/util/playback/Playable.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/util/playback/Playable.java
@@ -4,6 +4,7 @@ import android.content.Context;
 import android.content.SharedPreferences;
 import android.os.Parcelable;
 import android.preference.PreferenceManager;
+import android.support.annotation.Nullable;
 import android.util.Log;
 
 import java.util.List;
@@ -183,6 +184,7 @@ public interface Playable extends Parcelable,
          *
          * @return The restored Playable object
          */
+        @Nullable
         public static Playable createInstanceFromPreferences(Context context) {
             long currentlyPlayingMedia = PlaybackPreferences.getCurrentlyPlayingMedia();
             if (currentlyPlayingMedia != PlaybackPreferences.NO_MEDIA_PLAYING) {
diff --git a/core/src/main/java/de/danoeh/antennapod/core/util/playback/PlaybackController.java b/core/src/main/java/de/danoeh/antennapod/core/util/playback/PlaybackController.java
index d1d6b5c63..7c930a8e2 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/util/playback/PlaybackController.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/util/playback/PlaybackController.java
@@ -14,6 +14,7 @@ import android.os.Build;
 import android.os.IBinder;
 import android.preference.PreferenceManager;
 import android.support.annotation.NonNull;
+import android.support.annotation.Nullable;
 import android.support.v4.content.ContextCompat;
 import android.text.TextUtils;
 import android.util.Log;
@@ -212,25 +213,26 @@ public abstract class PlaybackController {
      * Returns an intent that starts the PlaybackService and plays the last
      * played media or null if no last played media could be found.
      */
-    private Intent getPlayLastPlayedMediaIntent() {
+    @Nullable private Intent getPlayLastPlayedMediaIntent() {
         Log.d(TAG, "Trying to restore last played media");
         Playable media = PlayableUtils.createInstanceFromPreferences(activity);
-        if (media != null) {
-            Intent serviceIntent = new Intent(activity, PlaybackService.class);
-            serviceIntent.putExtra(PlaybackService.EXTRA_PLAYABLE, media);
-            serviceIntent.putExtra(PlaybackService.EXTRA_START_WHEN_PREPARED, false);
-            serviceIntent.putExtra(PlaybackService.EXTRA_PREPARE_IMMEDIATELY, true);
-            boolean fileExists = media.localFileAvailable();
-            boolean lastIsStream = PlaybackPreferences.getCurrentEpisodeIsStream();
-            if (!fileExists && !lastIsStream && media instanceof FeedMedia) {
-                DBTasks.notifyMissingFeedMediaFile(activity, (FeedMedia) media);
-            }
-            serviceIntent.putExtra(PlaybackService.EXTRA_SHOULD_STREAM,
-                    lastIsStream || !fileExists);
-            return serviceIntent;
+        if (media == null) {
+            Log.d(TAG, "No last played media found");
+            return null;
         }
-        Log.d(TAG, "No last played media found");
-        return null;
+
+        Intent serviceIntent = new Intent(activity, PlaybackService.class);
+        serviceIntent.putExtra(PlaybackService.EXTRA_PLAYABLE, media);
+        serviceIntent.putExtra(PlaybackService.EXTRA_START_WHEN_PREPARED, false);
+        serviceIntent.putExtra(PlaybackService.EXTRA_PREPARE_IMMEDIATELY, true);
+        boolean fileExists = media.localFileAvailable();
+        boolean lastIsStream = PlaybackPreferences.getCurrentEpisodeIsStream();
+        if (!fileExists && !lastIsStream && media instanceof FeedMedia) {
+            DBTasks.notifyMissingFeedMediaFile(activity, (FeedMedia) media);
+        }
+        serviceIntent.putExtra(PlaybackService.EXTRA_SHOULD_STREAM,
+                lastIsStream || !fileExists);
+        return serviceIntent;
     }