-
Notifications
You must be signed in to change notification settings - Fork 843
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Log is very chatty - V/MediaPlayer-JNI, V/MediaPlayerNative - reopened #933
Comments
Please use code blocks to properly format your issue... |
@luanpotter is there anything we can do about it? I think that log is sent by the native layer right? |
Yes. I added a Logger PR as a first step, now we need to wire it on the android side to try to suppress these messages, but it's likely gonna be hacky, I tried this before and wasn't trivial. |
@erickzanardo I'm also facing the same issue. If you have solved this issue please guide me. |
Probably this would be solved by replacing the Android MediaPlayer with ExoPlayer which is more actively developed and brings more advantages. |
# Description - Replace Platform Method Channels (from platform -> to lib) with Event Channels - Combined Event stream for player (`PlayerEvent`) and global (`GlobalEvent`) events - Add methods `create`, `dispose` and `getEventStream` to `AudioplayersPlatformInterface` - Add method `getGlobalEventStream` to `GlobalAudioplayersPlatformInterface` - Add log stream `player.onLog()` and `AudioPlayer.global.onLog()` - Example for using Logger - Throw PlatformException on Method Error via Method Channel (lib -> platform) - Emit PlatformException on Event Error via Event Channel (platform -> lib) `player.eventStream.listen().onError()` and `AudioPlayer.global.eventStream.listen().onError()` - Add docs ## Breaking Change ### Migration instructions **audioplayers**: | Before | After | |---|---| | deprecated `AudioPlayer.global.changeLogLevel(LogLevel.info)` | `Logger.logLevel = LogLevel.info` | | deprecated `AudioPlayer.global.logLevel` | `Logger.logLevel` | | deprecated `AudioPlayer.global.log()` | `Logger.log()` or `Logger.error()` | | deprecated `AudioPlayer.global.info()` | `Logger.log()` | | deprecated `AudioPlayer.global.error()` | `Logger.error()` | | `ForPlayer<>` | _removed_ | **audioplayers_platform_interface**: | Before | After | |---|---| | `LogLevel` | _moved_ to `audioplayers` package | | `StreamsInterface` | _removed_ | | `ForPlayer<>` | _removed_ | ## Related Issues Closes #106 Closes #151 Closes #1266 Can handle error of #1260 Related to #208 and #933, but cannot hide Android MediaPlayer logs This would bypass: flutter/flutter#69103
Sorry, can you please reopen this ticket #902?
On both Samsung Galaxy S8 and Samsung Galaxy S10 I get full log, as in the first post #902 (comment)
On the Android Emulator API 27 8.1 with Google Play I only get the DRM log #902 (comment)
I have downloaded the whole project from https://github.com/luanpotter/audioplayers/
When pressing play on the first widget, I get a huge log on the real devices.
The text was updated successfully, but these errors were encountered: