Author: uau Date: Fri Mar 16 15:55:41 2007 New Revision: 22634 Modified: trunk/libmpdemux/demux_mkv.c Log: Matroska seeking fixes If a relative seek forward went past the last index position the Matroska demuxer did not seek to any index position. It did however set the mkv_d->skip_to_timecode variable which meant that the next fill_buffer() call would read from the current position until the target position (probably the end of the file). Fix this by changing the code to seek to the last index position if that is between the current and target positions. Also change backwards relative seek to accept an exactly matching index position (<= vs <) and reorganize the seeking conditionals to allow making the above change without turning the code into a complete mess. Modified: trunk/libmpdemux/demux_mkv.c ============================================================================== --- trunk/libmpdemux/demux_mkv.c (original) +++ trunk/libmpdemux/demux_mkv.c Fri Mar 16 15:55:41 2007 @@ -28,6 +28,8 @@ #include "libass/ass.h" #include "libass/ass_mp.h" +#include "libavutil/common.h" + #ifdef USE_QTX_CODECS #include "loader/qtx/qtxsdk/components.h" #endif @@ -3554,18 +3556,26 @@ demux_mkv_seek (demuxer_t *demuxer, floa diff = target_timecode + mkv_d->first_tc - (int64_t) mkv_d->indexes[i].timecode * mkv_d->tc_scale / 1000000.0; - if ((flags & 1 || target_timecode <= mkv_d->last_pts*1000) - && diff >= 0 && diff < min_diff) - { - min_diff = diff; - index = mkv_d->indexes + i; - } - else if (target_timecode > mkv_d->last_pts*1000 - && diff < 0 && -diff < min_diff) - { - min_diff = -diff; - index = mkv_d->indexes + i; - } + if ((flags & 1 || target_timecode <= mkv_d->last_pts*1000)) { + // Absolute seek or seek backward: find the last index + // position before target time + if (diff < 0 || diff >= min_diff) + continue; + } + else { + // Relative seek forward: find the first index position + // after target time. If no such index exists, find last + // position between current position and target time. + if (diff <= 0) { + if (min_diff <= 0 && diff <= min_diff) + continue; + } + else if (diff >= FFMIN(target_timecode - mkv_d->last_pts, + min_diff)) + continue; + } + min_diff = diff; + index = mkv_d->indexes + i; } if (index) /* We've found an entry. */