diff options
| author | wm4 <wm4@nowhere> | 2013-03-19 02:27:47 +0100 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2013-03-19 02:27:47 +0100 |
| commit | 9a731a9b0df80590a7808b8d0628b84d06c66005 (patch) | |
| tree | b7102f0113925ab83536e16e749d0abbb9ffeb7b /demux/asfheader.c | |
| parent | be7e04f7198ed6092a1d1e54e867df9b2c6247a4 (diff) | |
| download | mpv-9a731a9b0df80590a7808b8d0628b84d06c66005.tar.bz2 mpv-9a731a9b0df80590a7808b8d0628b84d06c66005.tar.xz | |
demux: fix regressions by restricting cover art hack further
The code modified by this commit is supposed to prevent demuxing the
whole file when cover art is present. (The problem with cover art is
that the ffmpeg libavformat API doesn't signal video EOF correctly - so
we try to read more packets to find the next video frame, which results
in demuxing and queuing the whole audio stream.)
This caused regressions for files with extremely high audio offset (see
github issue #46). MY conclusion is that this cover art crap doesn't
work, and this is just another case of completely insane ffmpeg/libav
API.
Disable the hack in all cases, unless a cover art video track is
selected. Maybe I'll handle cover art directly in the frontend later, so
that we don't have to rely on whatever libavformat does.
Unfortunately, this also makes behavior with equally insane mp4 files
with sparse video tracks worse, but this issue takes priority.
Diffstat (limited to 'demux/asfheader.c')
0 files changed, 0 insertions, 0 deletions