• Clément Bœsch's avatar
    microdvd: sanitize AVPackets. · 1f265f52
    Clément Bœsch authored
    Current MicroDVD AVPackets contain timing information and trailing line
    breaks. The data is now only composed of the markup data. Doing this
    consistently between text subtitles decoders allows to use different
    codec for various formats. For instance, MicroDVD markup is sometimes
    found in some VPlayer files. Also, generally speaking, the subtitles
    text decoders have no use of these timings (and they must not use them
    since it would break any user timing adjustment).
    
    Technically, this is a major ABI break. In practice, a mismatching
    lavf/lavc will now error out for MicroDVD decoding. Supporting both
    formats requires unnecessary complex and fragile code.
    
    FATE needs update because line breaks in the ASS file were "\n" (because
    that's what is used in the original file). ASS format expect "\r\n" line
    breaks; this commit fixes this issue. Also note that this "\r\n"
    trailing need to be moved at some point from the decoders to the ASS
    muxer.
    1f265f52
microdvddec.c 10.8 KB