fix: add @CanIgnoreReturnValue to avoid errorprone errors (#1716)

In the Google monorepo, this was blocking the build from succeeding. Fixes #1710.
googleapis · Sep 1, 2022 · cba2f82 · cba2f82
1 parent 9efcb14
commit cba2f82
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/google-http-client/pom.xml b/google-http-client/pom.xml
@@ -135,6 +135,10 @@
       <groupId>com.google.code.findbugs</groupId>
       <artifactId>jsr305</artifactId>
     </dependency>
+    <dependency>
+      <groupId>com.google.errorprone</groupId>
+      <artifactId>error_prone_annotations</artifactId>
+    </dependency>
     <dependency>
       <groupId>com.google.guava</groupId>
       <artifactId>guava</artifactId>

diff --git a/google-http-client/src/main/java/com/google/api/client/util/SslUtils.java b/google-http-client/src/main/java/com/google/api/client/util/SslUtils.java
@@ -14,6 +14,7 @@
 
 package com.google.api.client.util;
 
+import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import java.security.GeneralSecurityException;
 import java.security.KeyStore;
 import java.security.NoSuchAlgorithmException;
@@ -101,6 +102,7 @@ public static KeyManagerFactory getPkixKeyManagerFactory() throws NoSuchAlgorith
    *     #getPkixTrustManagerFactory()})
    * @since 1.14
    */
+  @CanIgnoreReturnValue
   public static SSLContext initSslContext(
       SSLContext sslContext, KeyStore trustStore, TrustManagerFactory trustManagerFactory)
       throws GeneralSecurityException {