Enable Clippy and fix warnings

- Drop unnecessary ampersands
- Use `.is_empty()` rather than `.len()` comparisons
- Don't `use` top-level imports
- Constants automatically have the `'static` lifetime
- Use `matches!` macro
- Don't cast `as` the same type
- Drop unnecessary `return`
- Drop unnecessary 1-arg closure in `.map()`
- Elide unncessary lifetimes

Bug: 342362349
Test: build
Change-Id: I59c55d41a770a912a74eebf0ef6f3b317c381c2a
diff --git a/keys.rs b/keys.rs
index 84197cf..8862a33 100644
--- a/keys.rs
+++ b/keys.rs
@@ -33,11 +33,11 @@
 /// Key slot identification; matches the value used in
 /// `OpenSSLKeymasterEnforcement::GetKeyAgreementKey` in `openssl_keymaster_enforcement.cpp` for
 /// back-compatibility.
-const KM_KAK_SLOT_ID: &'static [u8] = b"com.android.trusty.keymint.kak\0";
+const KM_KAK_SLOT_ID: &[u8] = b"com.android.trusty.keymint.kak\0";
 
 /// Key derivation input data; matches `kMasterKeyDerivationData` in `trusty_keymaster_context.cpp`
 /// for back-compatibility.
-const KM_KEY_DERIVATION_DATA: &'static [u8] = b"KeymasterMaster\0";
+const KM_KEY_DERIVATION_DATA: &[u8] = b"KeymasterMaster\0";
 
 /// Size of a `u32` value in bytes.
 const U32_SIZE: usize = core::mem::size_of::<u32>();