[FFmpeg-devel] [PATCH v6 0/4] Initial implementation of TTML encoding/muxing

[Jan Ekström]

On Thu, Mar 4, 2021, 16:43 Moritz Barsnick <barsnick at gmx.net> wrote:

> On Thu, Mar 04, 2021 at 15:52:13 +0200, Jan Ekström wrote:
> > I... just am not so sure in this case splitting the TTML writer and
> > the TTML encoder from each other makes sense.
>
> I agree. In very many cases, muxer and encoder, or demuxer and decoder
> are only useful together.
>
> Yet I have observed that commits to libavformat and libavcodec are
> usually separated. Check the addition of many other such combinations.
> (I think usually the codec comes first, because the muxer/demuxer uses
> the codec definition.
>

Yes, I have looked at things, and I after I go for a walk will specifically
check previous subtitle additions. Meanwhile, please take a look at the
alternatives that I've raised and please tell me of either of those makes
a) sense b) it easier to revert things in case that is wanted.

The codec definition is there already, as demuxing of ttml packets from mxf
was already implemented. That - as I did note in my previous message would
be why if you really wanted the muxer could be brought in first as well.

Jan