search

martinbonnin / ExoPlayer

Apache License 2.0
22 stars 3 forks source link

Hls merge with updated google #35

Closed bhnath closed 9 years ago

bhnath commented 10 years ago

There have been some updates to the ExoPlayer on google's github tree. I have gone ahead and merged those changes into the raw hls branch (as part of my own effort to add ID3 tag support for HLS streams which can be contributed next). After these changes are merged the master branch of this tree could be updated to the latest from Google.

martinbonnin commented 10 years ago

Just to make sure I understand: this corresponds to merging the master of google/Exoplayer into the hls branch of martinbonnin/Exoplayer ?

bhnath commented 10 years ago

Correct. There were some merge conflicts to be resolved when I pulled the latest master of google/ExoPlayer and this should remedy that.

Have you seen the new dev-hls branch under google/ExoPlayer?

perchrh commented 10 years ago

I get a critical error on every stream after applying this code. It happens on stream startup. As a result, no content plays. This occurs even with clean build and install of the bhnath:hls_merge_with_updated_google branch.

10-14 14:44:41.198 21309-21329/com.google.android.exoplayer.demo E/ExoPlayerImplInternal﹕ Internal runtime error. java.lang.NullPointerException at com.google.android.exoplayer.hls.HLSSampleSource.readData(HLSSampleSource.java:540) at com.google.android.exoplayer.MediaCodecTrackRenderer.feedInputBuffer(MediaCodecTrackRenderer.java:520) at com.google.android.exoplayer.MediaCodecTrackRenderer.doSomeWork(MediaCodecTrackRenderer.java:417) at com.google.android.exoplayer.MediaCodecAudioTrackRenderer.doSomeWork(MediaCodecAudioTrackRenderer.java:273) at com.google.android.exoplayer.ExoPlayerImplInternal.doSomeWork(ExoPlayerImplInternal.java:421) at com.google.android.exoplayer.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:203) at android.os.Handler.dispatchMessage(Handler.java:98) at android.os.Looper.loop(Looper.java:136) at android.os.HandlerThread.run(HandlerThread.java:61) at com.google.android.exoplayer.ExoPlayerImplInternal$1.run(ExoPlayerImplInternal.java:109)

martinbonnin commented 9 years ago

Code has changed so much recently, I assume this is not relevant anymore