> On Feb 23, 2023, at 04:17, TADANO Tokumei <aiming...@pc.nifty.jp> wrote: > > Ping with rebased patch set. > Some chages are added to reflect review results from outside of this ML. > > --- > This patch set add another ARIB caption decoder using libaribcaption > external library: https://github.com/xqq/libaribcaption > The library decodes subtitles of ISDB-based TV broadcasting. > It is not only for Japanese ARIB STD-B24 caption, but also for > Brazilian ABNT NBR 15606-1 and Philippines version of ISDB-T. > > Unlike libaribb24, it supports 3 types of subtitle outputs: > * text: plain text > * ass: ASS formatted text > * bitmap: bitmap image > > Default subtitle type is ASS as same as libaribb24. > Advantages compared with libaribb24 on ASS subtitle are: > * Subtitle positioning. > * Multi-rect subtitle: some captions are displayed at different > position at a time. > * More stability and reproducibility. > > Subtitle type can be changed by `-sub_type` option. > You can see ARIB caption with ffplay tool: > ffplay -sub_type bitmap MPEG.TS > > Sample files exist under: > https://streams.videolan.org/streams/ts/ARIB/ > Some of them are encrypted and some don't contain ARIB caption data. > Good samples for ARIB caption are: > brazil/07-25_20-33-35_UCV - HD_.ts > japan/channel2[137]_clear.ts > japan/osaka_15.ts > > --- > v7: reflect review results from outside of this ML > - rename -ass_workaround option to -ass_single_rect and default to > false (latest mpv supports multiple ASS rectangle) > - add -canvas_size option to specify frame size to render bitmap subtitle > - add -caption_encoding option to specify encoding of subtitle text > - change behavior of profile C ARIB caption > - some cosmetic changes > v6: reflect review results from outside of this ML > - check clut table overflow > - do not adjust vertical position of ruby for ASS format > v5: reflect review comments from Mao Hata > - reset correct variable in aribcaption_close() > - add aribcaption_close() to some places in aribcaption_init() > - check if av_strdup() returns NULL in set_ass_header() > v4: reflect review results from outside of this ML. > - resize bitmap subtitle image to display fonts with correct aspect ratio > - multiple font families can be specified by '-font' option > - remove 'rendering_backend' option > - add document > - minor bug fixes > v3: combine former 1/4 and 2/4 due to the patchwork shows build error. > - fix help option content which incorrectly separated to 2 lines in 2/4. > - amend commit message of 4/4. > > TADANO Tokumei (4): > lavc/libaribcaption.c: add ARIB caption decoder using libaribcaption > lavc/codec_desc.c: remove AV_CODEC_PROP_TEXT_SUB property > lavf/mpegts.c: set some properties for ARIB caption > doc/decoders.texi: add document of aribcaption decoder > > configure | 4 + > doc/decoders.texi | 150 +++++ > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/codec_desc.c | 1 - > libavcodec/libaribb24.c | 1 + > libavcodec/libaribcaption.c | 1171 +++++++++++++++++++++++++++++++++++ > libavformat/mpegts.c | 6 +- > 8 files changed, 1333 insertions(+), 2 deletions(-) > create mode 100644 libavcodec/libaribcaption.c > > -- > 2.30.2 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
This set looks reasonable to me at this point; the one thing blocking it is the issue with assenc.c muxing for AVSubtitles with multiple rects, which needs a decision on this question: http://ffmpeg.org/pipermail/ffmpeg-devel/2023-February/306850.html Once that's resolved, I'm happy to merge this. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
