[Android] Code cleaning and safety checks
Geoffrey Métais
git at videolan.org
Thu Sep 12 16:34:31 CEST 2019
vlc-android | branch: master | Geoffrey Métais <geoffrey.metais at gmail.com> | Thu Sep 12 11:51:47 2019 +0200| [3b478388cf6ca45523a8aa1bb2fb439f1b20d139] | committer: Geoffrey Métais
Code cleaning and safety checks
(cherry picked from commit 6bc276383cd1d63ac664ed7d03cd93397ddb50e5)
> https://code.videolan.org/videolan/vlc-android/commit/3b478388cf6ca45523a8aa1bb2fb439f1b20d139
---
vlc-android/src/org/videolan/vlc/PlaybackService.kt | 7 +------
vlc-android/src/org/videolan/vlc/StartActivity.kt | 4 ++--
2 files changed, 3 insertions(+), 8 deletions(-)
diff --git a/vlc-android/src/org/videolan/vlc/PlaybackService.kt b/vlc-android/src/org/videolan/vlc/PlaybackService.kt
index d7264fbdb..55181b882 100644
--- a/vlc-android/src/org/videolan/vlc/PlaybackService.kt
+++ b/vlc-android/src/org/videolan/vlc/PlaybackService.kt
@@ -1303,13 +1303,8 @@ class PlaybackService : MediaBrowserServiceCompat(), CoroutineScope, LifecycleOw
ContextCompat.startForegroundService(context, serviceIntent)
}
- fun getService(iBinder: IBinder): PlaybackService? {
- val binder = iBinder as LocalBinder
- return binder.service
- }
-
fun loadLastAudio(context: Context) {
- val i = Intent(context, PlaybackService::class.java).apply { action = ACTION_REMOTE_LAST_PLAYLIST }
+ val i = Intent(ACTION_REMOTE_LAST_PLAYLIST, null, context, PlaybackService::class.java)
ContextCompat.startForegroundService(context, i)
}
diff --git a/vlc-android/src/org/videolan/vlc/StartActivity.kt b/vlc-android/src/org/videolan/vlc/StartActivity.kt
index f431f6113..cd36defe4 100644
--- a/vlc-android/src/org/videolan/vlc/StartActivity.kt
+++ b/vlc-android/src/org/videolan/vlc/StartActivity.kt
@@ -196,10 +196,10 @@ class StartActivity : FragmentActivity() {
}
private fun startPlaybackFromApp(intent: Intent) {
- if (intent.type != null && intent.type!!.startsWith("video"))
+ if (intent.type?.startsWith("video") == true)
startActivity(intent.setClass(this, VideoPlayerActivity::class.java))
else
- MediaUtils.openMediaNoUi(intent.data!!)
+ intent.data.let { MediaUtils.openMediaNoUi(it) }
finish()
}
More information about the Android
mailing list