From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id DD94444D52 for ; Thu, 23 Mar 2023 09:11:08 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3CAA768C223; Thu, 23 Mar 2023 11:11:05 +0200 (EET) Received: from conssluserg-06.nifty.com (conssluserg-06.nifty.com [210.131.2.91]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 131A268C18F for ; Thu, 23 Mar 2023 11:10:56 +0200 (EET) Received: from [192.168.0.9] (M106073025034.v4.enabler.ne.jp [106.73.25.34]) (authenticated) by conssluserg-06.nifty.com with ESMTP id 32N9AWOR032060 for ; Thu, 23 Mar 2023 18:10:35 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-06.nifty.com 32N9AWOR032060 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1679562635; bh=n5XuLgc7LlQa7L4Zf85E1Nl4/V2XpSijjWKqtIoTJBI=; h=Date:From:Subject:To:References:In-Reply-To:From; b=B9M+zfOsDm55vY+coK4gemV7r/tVNjlp5dXpejIHw97siqUArbZSHCXx0io3RxXeW uTth41z/TOP4Ihb7jytiWaOuw3kUMwWAeyc80nqgkEEEAZ9CDJ0xUvlaf0XOYqzaR/ b8t19V5qaQA4vcOJGSsWmZkl6qXKMniKpB4ge0OZWeM/mLv2uolFgb8LATz8ncMmjK uDEMaBIvxNpqqLAuLJLatTPJyzWxxBIjBpl9n+MPzEZv4NGODv4hbRP2dO2wLX7O2m Sj5mJUVrlbFFz8a5hb63Kdis7+krMuW3gsZd0swN0RTRG1beUqOflmP2bZHh68eyJR 694MFFoEvZwkw== X-Nifty-SrcIP: [106.73.25.34] Message-ID: <2fb0fbb9-4844-e22a-8fc8-d8bb97d805e8@pc.nifty.jp> Date: Thu, 23 Mar 2023 18:10:31 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 From: TADANO Tokumei To: ffmpeg-devel@ffmpeg.org References: <20230223101718.562202-1-aimingoff@pc.nifty.jp> Content-Language: en-US In-Reply-To: <20230223101718.562202-1-aimingoff@pc.nifty.jp> Subject: Re: [FFmpeg-devel] [PATCH v7 0/4] add ARIB caption decoder using libaribcaption X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 2023/02/23 19:17, TADANO Tokumei 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 > ping! Are there any other comments? If nobody has objections, could anyone push the patch set? _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".