From 06bcf75f9b95000b83a00ca5842f40e0e09cfd10 Mon Sep 17 00:00:00 2001 From: Hans-Christoph Steiner Date: Thu, 6 Jul 2017 23:07:23 +0200 Subject: [PATCH] put the faster check first in App.isInstalled() A > is much faster than all the stuff that isMediaInstalled() has to do, so put > first, so that if it is true, isMediaInstalled() is not executed. --- app/src/main/java/org/fdroid/fdroid/data/App.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/java/org/fdroid/fdroid/data/App.java b/app/src/main/java/org/fdroid/fdroid/data/App.java index 58f209a36..49cc65a02 100644 --- a/app/src/main/java/org/fdroid/fdroid/data/App.java +++ b/app/src/main/java/org/fdroid/fdroid/data/App.java @@ -856,7 +856,7 @@ public class App extends ValueObject implements Comparable, Parcelable { } public boolean isInstalled(Context context) { - return isMediaInstalled(context) || installedVersionCode > 0; + return installedVersionCode > 0 || isMediaInstalled(context); } public boolean isMediaInstalled(Context context) {