diff mbox series

[FFmpeg-devel] doc/bitstream_filters: add filter_units practical examples for removing closed captions

Message ID 20231209193112.783495-1-marth64@proxyid.net
State New
Headers show
Series [FFmpeg-devel] doc/bitstream_filters: add filter_units practical examples for removing closed captions | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Marth64 Dec. 9, 2023, 7:31 p.m. UTC
---
 doc/bitstream_filters.texi | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Comments

Stefano Sabatini Dec. 10, 2023, 12:53 a.m. UTC | #1
On date Saturday 2023-12-09 13:31:13 -0600, Marth64 wrote:
> ---
>  doc/bitstream_filters.texi | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
> index 145aa8684e..b01ede0c93 100644
> --- a/doc/bitstream_filters.texi
> +++ b/doc/bitstream_filters.texi
> @@ -213,6 +213,23 @@ To remove all AUDs, SEI and filler from an H.265 stream:
>  ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT
>  @end example
>  

> +By nature of removing SEI messages, the filter can also remove Closed Captions.

I think you might expand this to shortly explain the relation between
Closed Captions and SEI message. Also, I'd avoid the expression "by
nature" and replace with a plainer/more explicit expression (as most
of the readers are not English speakers).

[...]
Marth64 Dec. 10, 2023, 1:26 a.m. UTC | #2
Appreciate the good feedback, will update. Thank you.

On Sat, Dec 9, 2023 at 6:53 PM Stefano Sabatini <stefasab@gmail.com> wrote:

> On date Saturday 2023-12-09 13:31:13 -0600, Marth64 wrote:
> > ---
> >  doc/bitstream_filters.texi | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> >
> > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
> > index 145aa8684e..b01ede0c93 100644
> > --- a/doc/bitstream_filters.texi
> > +++ b/doc/bitstream_filters.texi
> > @@ -213,6 +213,23 @@ To remove all AUDs, SEI and filler from an H.265
> stream:
> >  ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40'
> OUTPUT
> >  @end example
> >
>
> > +By nature of removing SEI messages, the filter can also remove Closed
> Captions.
>
> I think you might expand this to shortly explain the relation between
> Closed Captions and SEI message. Also, I'd avoid the expression "by
> nature" and replace with a plainer/more explicit expression (as most
> of the readers are not English speakers).
>
> [...]
>
diff mbox series

Patch

diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
index 145aa8684e..b01ede0c93 100644
--- a/doc/bitstream_filters.texi
+++ b/doc/bitstream_filters.texi
@@ -213,6 +213,23 @@  To remove all AUDs, SEI and filler from an H.265 stream:
 ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT
 @end example
 
+By nature of removing SEI messages, the filter can also remove Closed Captions.
+
+To remove SEI messages (including CC) from a MPEG-2 stream:
+@example
+ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT
+@end example
+
+To remove SEI messages (including CC) from a H264 stream:
+@example
+ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT
+@end example
+
+To remove SEI messages (including CC and dynamic HDR) from a HEVC stream:
+@example
+ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39' OUTPUT
+@end example
+
 @section hapqa_extract
 
 Extract Rgb or Alpha part of an HAPQA file, without recompression, in order to create an HAPQ or an HAPAlphaOnly file.