diff options
| author | Carl Hetherington <cth@carlh.net> | 2015-10-16 14:38:44 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2015-10-16 14:38:44 +0100 |
| commit | 9e025d3f85f9d6d855b3d5e6c90bca0eac3a3d49 (patch) | |
| tree | afc3e4a093a357bc7144a554c139e71b05fca9c5 /src/lib/subtitle_decoder.cc | |
| parent | e7811b466eff496db7f63842df2fa4a4410afe14 (diff) | |
Avoid decoding other packets when looking for subs.
The "accumulation" of, for example, video data when we are looking
for audio data is an *optimisation* to reduce the number of seeks.
It should not be necessary for correctness (the output should be right
even if we never kept anything except what we were looking for).
Doing this accumulation is not always an optimisation; sometimes not
doing it is better. Avoiding it when going back for subtitles is one
of these cases.
Diffstat (limited to 'src/lib/subtitle_decoder.cc')
| -rw-r--r-- | src/lib/subtitle_decoder.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/subtitle_decoder.cc b/src/lib/subtitle_decoder.cc index a95964f47..30b8661ae 100644 --- a/src/lib/subtitle_decoder.cc +++ b/src/lib/subtitle_decoder.cc @@ -79,7 +79,7 @@ SubtitleDecoder::get (list<T> const & subs, list<ContentTimePeriod> const & sp, * (a) give us what we want, or * (b) hit the end of the decoder. */ - while (!pass () && (subs.empty() || (subs.back().period().to < sp.back().to))) {} + while (!pass(PASS_REASON_SUBTITLE) && (subs.empty() || (subs.back().period().to < sp.back().to))) {} /* Now look for what we wanted in the data we have collected */ /* XXX: inefficient */ |