dcpomatic/src/lib/subrip_decoder.cc, branch v2.4.12 DCP-o-matic DCP tools https://git.carlh.net/cgit/dcpomatic/atom?h=v2.4.12 2015-10-16T13:38:44Z Avoid decoding other packets when looking for subs. 2015-10-16T13:38:44Z Carl Hetherington cth@carlh.net 2015-10-16T13:38:44Z urn:sha1:9e025d3f85f9d6d855b3d5e6c90bca0eac3a3d49 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. Fix invalid Interop subtitle times in some cases. 2015-10-15T08:13:33Z Carl Hetherington cth@carlh.net 2015-10-15T08:13:33Z urn:sha1:6708d52b5c2ff9ccb6ee99f3b793d791bb95f26b Use a much higher "timecode rate" (1000 rather than 24) when representing the time of subtitles internally so that accuracy is preserved and we don't end up rounding things up to give out of range results. Lots of #include <iostream>s for Arch. 2015-09-14T21:49:36Z Carl Hetherington cth@carlh.net 2015-09-14T21:49:36Z urn:sha1:1858190cff2f960f3d1f0a5cc02c69da86088f5b Pass colour on when decoding. 2015-07-02T12:39:19Z Carl Hetherington cth@carlh.net 2015-07-02T12:39:19Z urn:sha1:88d2e9b659a7517c91a630c1cd67cc919c97d59a Fix missing subtitles in some cases. 2015-06-26T21:02:00Z Carl Hetherington cth@carlh.net 2015-06-26T21:02:00Z urn:sha1:c8ff422a42eac30517a7acde57ab84e55449f4e4 We were passing subtitles back from decoders to SubtitleDecoder using dcp::SubtitleStrings and relying on their storage of time to know when the subtitles were. These times are quantised (by the use of dcp::SubtitleString) and then compared with unquantised times (kept as ContentTime) in the main checking loop in SubtitleDecoder::get(). Fix this by storing periods as ContentTimePeriod as well as in the dcp::SubtitleStrings. No-op: remove all trailing whitespace. 2015-06-21T00:15:32Z Carl Hetherington cth@carlh.net 2015-06-21T00:15:32Z urn:sha1:e60bb3e51bd1508b149e6b8f6608f09b5196ae26 Remove PassReason stuff. 2015-06-12T12:47:35Z Carl Hetherington cth@carlh.net 2015-06-12T12:47:35Z urn:sha1:bdbfd6b08856a445446bfd845f1c43109d184250 This feels wrong: it means that it is possible for FFmpegDecoder to discard packets. I can't see how this is ok in all cases: maybe we were lucky that it worked at all. Fix horizontal positioning with .srt / burn-in (#488). 2015-06-03T23:17:12Z Carl Hetherington cth@carlh.net 2015-06-03T23:17:12Z urn:sha1:4f1751a9e43e515be773ee3871150f7beca325f5 Various fixes to allow independent x and y scaling of text subs (#489). 2015-06-03T21:54:17Z Carl Hetherington cth@carlh.net 2015-06-03T21:54:17Z urn:sha1:7affca9f7b210c3137eb084cd393b534051a2647 Don't bother decoding video frames when we're seeking around trying to find subtitles. 2015-05-13T09:15:26Z Carl Hetherington cth@carlh.net 2015-05-13T09:15:26Z urn:sha1:9922c1f2eaea674ba2ff6cce5f5853655fd8ad7a