From 0b1940c4bfc683c1cbfb7ab54aa34f48a867e405 Mon Sep 17 00:00:00 2001
From: Peter Serwylo <peter@serwylo.com>
Date: Fri, 24 Feb 2017 10:29:01 +1100
Subject: [PATCH] Make it explicit that it may return null.

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.
---
 .../main/java/org/fdroid/fdroid/AppUpdateStatusManager.java    | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/app/src/main/java/org/fdroid/fdroid/AppUpdateStatusManager.java b/app/src/main/java/org/fdroid/fdroid/AppUpdateStatusManager.java
index bdac704cd..90f00c7ad 100644
--- a/app/src/main/java/org/fdroid/fdroid/AppUpdateStatusManager.java
+++ b/app/src/main/java/org/fdroid/fdroid/AppUpdateStatusManager.java
@@ -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);