From 4be7e475840efd9739aaa910a68671912edb2afd Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Wed, 3 Jun 2026 09:38:24 -0400 Subject: [PATCH] chore(android): drop redundant !! on PlaylistRef.systemVariant MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Line 264 already null-checks playlist.systemVariant in the if condition. PlaylistRef is a data class with a val backing field, so the smart cast narrows it to String inside the branch — the !! on line 265 was a no-op the Kotlin compiler was warning about. --- .../main/java/com/fabledsword/minstrel/home/ui/HomeScreen.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/home/ui/HomeScreen.kt b/android/app/src/main/java/com/fabledsword/minstrel/home/ui/HomeScreen.kt index 63467dad..cf5a3649 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/home/ui/HomeScreen.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/home/ui/HomeScreen.kt @@ -262,7 +262,7 @@ class HomeViewModel @Inject constructor( val detail = try { withTimeout(PLAYLIST_FETCH_TIMEOUT_MS) { if (playlist.refreshable && playlist.systemVariant != null) { - playlistsRepository.systemShuffle(playlist.systemVariant!!) + playlistsRepository.systemShuffle(playlist.systemVariant) } else { playlistsRepository.refreshDetail(playlist.id) }