[FFmpeg-devel] [updated PATCH] Make srtenc accept text-only (type SUBTITLE_TEXT) subtitles

[Wolfram Gloger]

Hi,

Thanks for your comment.

Clément Bœsch <ubitux at gmail.com> writes:

> Currently, we make all the text decoders output to SUBTITLE_ASS (the
> common internal format), and actually it seems no subtitles text encoder
> will deal very well with SUBTITLE_TEXT (no git grep match all over the
> code base). Is it too much pain to make the decoder output ASS? See
> lavc/subviewerdec.c for a simple example using the ASS helpers.

Ok, will try.

I am not sure my TV supports ASS markup, though.
With plain UTF8 text, it surely works well.

> A more correct solution IMO would be to add some SUBTITLE_TEXT <->
> SUBTITLE_ASS conversion code before sending them to the encoder depending
> on its capabilities, since I'm not very happy about requiring the encoders
> to support multiple types of input just for encoding "text" subtitles.

I am certainly not requiring all encoders to support plain text.
In srtenc it is just a few LOC so I still think it's ok.

Regards,
Wolfram.