chatrealm / DctvAndroid

Android app for http://diamondclub.tv
5 stars 5 forks source link

switch video player library to ijkplayer #17

Open kevbob1 opened 7 years ago

kevbob1 commented 7 years ago

https://github.com/Bilibili/ijkplayer

  1. it is open source
  2. under active development
  3. distributed as jars (no bundling in app)
  4. works...?
RenAigu commented 7 years ago

Since the stream url resolver works fine, I guess ijkplayer or Vitamio is not necessary, so I got Exoplayer (via an ExoMedia wrapper) working. On my device I get smoother playback and faster loading. I removed the flags that emulate 32 bit on 64 bit devices (the workaround to fix earlier errors), so performance on those devices should increase massively as well. @Excape can you test the newest commit if no errors reappear?

kevbob1 commented 7 years ago

+1 there is alot of buzz around ExoPlayer, I watched a few tutorials and it looked promising. I'm glad to move off Vitamio.

Good job.


Kevin

On Tue, Dec 20, 2016 at 11:25 AM, RenAigu notifications@github.com wrote:

Since the stream url resolver works fine, I guess ijkplayer or Vitamio is not necessary, so I got Exoplayer (via an ExoMedia wrapper) working. On my device I get smoother playback and faster loading. I removed the flags that emulate 32 bit on 64 bit devices (the workaround to fix earlier errors), so performance on those devices should increase massively as well. @Excape https://github.com/Excape can you test the newest commit if no errors reappear?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/chatrealm/DctvAndroid/issues/17#issuecomment-268318747, or mute the thread https://github.com/notifications/unsubscribe-auth/AATovRkmsQasefssbrKPvwdhzPihPCqGks5rKB2wgaJpZM4KzWmN .

RenAigu commented 7 years ago

Confirmed Nexus 6p working with someone in chat, so I'm pushing the update.

Excape commented 7 years ago

Can confirm, I tested it on a Nexus 6p and it works really well. Great work @RenAigu !