From 21c1d715514cde291f935c8c2cfccbbb44e3c343 Mon Sep 17 00:00:00 2001 From: Jonathan Klabunde Tomer Date: Thu, 24 Apr 2025 15:27:17 -0700 Subject: [PATCH] take advantage of list non-nullitude --- .../textsecuregcm/controllers/KeysController.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/controllers/KeysController.java b/service/src/main/java/org/whispersystems/textsecuregcm/controllers/KeysController.java index 13020edc9..8b59bc172 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/controllers/KeysController.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/controllers/KeysController.java @@ -152,7 +152,7 @@ public class KeysController { final List> storeFutures = new ArrayList<>(4); - if (setKeysRequest.preKeys() != null && !setKeysRequest.preKeys().isEmpty()) { + if (!setKeysRequest.preKeys().isEmpty()) { Metrics.counter(STORE_KEYS_COUNTER_NAME, Tags.of(platformTag, primaryDeviceTag, identityTypeTag, Tag.of(KEY_TYPE_TAG_NAME, "ec"))) .increment(); @@ -168,7 +168,7 @@ public class KeysController { storeFutures.add(keysManager.storeEcSignedPreKeys(identifier, device.getId(), setKeysRequest.signedPreKey())); } - if (setKeysRequest.pqPreKeys() != null && !setKeysRequest.pqPreKeys().isEmpty()) { + if (!setKeysRequest.pqPreKeys().isEmpty()) { Metrics.counter(STORE_KEYS_COUNTER_NAME, Tags.of(platformTag, primaryDeviceTag, identityTypeTag, Tag.of(KEY_TYPE_TAG_NAME, "kyber"))) .increment(); @@ -192,11 +192,7 @@ public class KeysController { final IdentityKey identityKey, @Nullable final String userAgent) { - final List> signedPreKeys = new ArrayList<>(); - - if (setKeysRequest.pqPreKeys() != null) { - signedPreKeys.addAll(setKeysRequest.pqPreKeys()); - } + final List> signedPreKeys = new ArrayList<>(setKeysRequest.pqPreKeys()); if (setKeysRequest.pqLastResortPreKey() != null) { signedPreKeys.add(setKeysRequest.pqLastResortPreKey());