Karl/BobStore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make it explicit that it may return null.

Browse Source 
Given that some places where this method is invoked does a null check,
it seems reasonable to make this assumption explicit. That way if devs
use the method in the future, then they will be aware of the contract.
This commit is contained in:
Peter Serwylo 2017-02-24 10:29:01 +11:00 committed by mvp76
parent 875b0d091f
commit 0b1940c4bf
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/src/main/java/org/fdroid/fdroid/AppUpdateStatusManager.java
View File

@ -6,6 +6,7 @@ import android.content.ContentResolver;
import android.content.Context;
import android.content.Intent;
import android.content.pm.PackageManager;
import android.support.annotation.Nullable;
import android.support.v4.app.TaskStackBuilder;
import android.support.v4.content.LocalBroadcastManager;
@ -76,6 +77,7 @@ public class AppUpdateStatusManager {
localBroadcastManager = LocalBroadcastManager.getInstance(context.getApplicationContext());
}
@Nullable
public AppUpdateStatus get(String key) {
synchronized (appMapping) {
return appMapping.get(key);
@ -204,6 +206,7 @@ public class AppUpdateStatusManager {
}
}
@Nullable
public Apk getApk(String key) {
synchronized (appMapping) {
AppUpdateStatus entry = appMapping.get(key);