commit 16110f320dd5b6a40af87eaba4bf3af60cb0efd1
author Edwin Kempin <ekempin@google.com> Wed Oct 12 13:09:13 2022 +0200
committer Edwin Kempin <ekempin@google.com> Wed Oct 12 13:09:13 2022 +0200
tree 3a0ba81e007f15fa7ef2ef508a99408ed4c406c2
parent d8815bf9660b6655696db242b8ad2801e866c036

Annotate methods that return a definitely null value with @Nullable

We intend to enable the ReturnMissingNullable errorprone pattern in
Gerrit core and this fixes the current issues.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I53a642afe69214799324f572c8d8737abc88c986

src/main/java/com/googlesource/gerrit/plugins/webhooks/DefaultHttpClientProvider.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/webhooks/DefaultHttpClientProvider.java b/src/main/java/com/googlesource/gerrit/plugins/webhooks/DefaultHttpClientProvider.java
index cd034ad..1eabb20 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/webhooks/DefaultHttpClientProvider.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/webhooks/DefaultHttpClientProvider.java
@@ -15,6 +15,7 @@
 package com.googlesource.gerrit.plugins.webhooks;
 
 import com.google.common.flogger.FluentLogger;
+import com.google.gerrit.common.Nullable;
 import com.google.inject.Inject;
 import com.google.inject.Provider;
 import java.security.KeyManagementException;
@@ -51,6 +52,7 @@
             .build();
   }
 
+  @Nullable
   private static SSLContext buildSslContext() {
     try {
       TrustManager[] trustAllCerts = new TrustManager[] {new DummyX509TrustManager()};