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

[Marth64]

Appreciate the good feedback, will update. Thank you.

On Sat, Dec 9, 2023 at 6:53 PM Stefano Sabatini <stefasab at 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).
>
> [...]
>