• Anton Khirnov's avatar
    fftools/ffmpeg: always use the same path for setting InputStream.[next_]dts · 129c7bf5
    Anton Khirnov authored
    Currently those are set in different ways depending on whether the
    stream is decoded or not, using some values from the decoder if it is.
    This is wrong, because there may be arbitrary amount of delay between
    input packets and output frames (depending e.g. on the thread count when
    frame threading is used).
    
    Always use the path that was previously used only for streamcopy. This
    should not cause any issues, because these values are now used only for
    streamcopy and discontinuity handling.
    
    This change will allow to decouple discontinuity processing from
    decoding and move it to ffmpeg_demux. It also makes the code simpler.
    
    Changes output in fate-cover-art-aiff-id3v2-remux and
    fate-cover-art-mp3-id3v2-remux, where attached pictures are now written
    in the correct order. This happens because InputStream.dts is no longer
    reset to AV_NOPTS_VALUE after decoding, so streamcopy actually sees
    valid dts values.
    129c7bf5
cover-art-aiff-id3v2-remux 1.78 KB