From 2e050a5c98b0f9bfd1ada28ded96d74c64536612 Mon Sep 17 00:00:00 2001 From: Luca Stefani Date: Mon, 3 Jan 2022 13:16:16 +0100 Subject: [PATCH] settings: Move one if check to else if The map is always empty if the previous check was true Change-Id: Id1f939d112789ab3c7da711636b40c8a4c23d507 --- sdk/src/java/lineageos/providers/LineageSettings.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sdk/src/java/lineageos/providers/LineageSettings.java b/sdk/src/java/lineageos/providers/LineageSettings.java index e8a75d8f..ecd4039a 100644 --- a/sdk/src/java/lineageos/providers/LineageSettings.java +++ b/sdk/src/java/lineageos/providers/LineageSettings.java @@ -270,9 +270,7 @@ public final class LineageSettings { mValues.clear(); mValuesVersion = newValuesVersion; - } - - if (mValues.containsKey(name)) { + } else if (mValues.containsKey(name)) { return mValues.get(name); // Could be null, that's OK -- negative caching } }