[MPlayer-dev-eng] [PATCH] Fix MicroDVD color tag

Reimar Döffinger Reimar.Doeffinger at gmx.de
Mon Nov 24 16:21:50 CET 2014


On 23.11.2014, at 11:51, Michal Wazgird <mplayer at cyryl.net> wrote:
> On Sat, Nov 22, 2014 at 6:04 PM, compn <tempn at mi.rr.com> wrote:
> 
>> On Sat, 22 Nov 2014 15:53:14 +0100
>> Michal Wazgird <mplayer at cyryl.net> wrote:
>> 
>>> This is my first patch submission, so sorry for any misbehaviour :)
>> 
>> you forgot to attach it! :)
>> 
> 
> Sorry :)
> This time with attachment. And to proper mail :)

Thanks, but I prefer the way this was applied in FFmpeg, which just skips any and all $ and # at the start.
You patch would break any subtitles that do not start with a $.
I don't mind if you want it to warn about subtitles that do not match the specification, but I'd like us to accept them anyway.


More information about the MPlayer-dev-eng mailing list