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

From: Marth64 <marth64@proxyid.net>
To: FFmpeg development discussions and patches
	<ffmpeg-devel@ffmpeg.org>, Marth64 <marth64@proxyid.net>
Subject: Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
Date: Sat, 9 Dec 2023 19:26:34 -0600
Message-ID: <CA+28BfD1SCFQr-u9-fEyse3=BzDs3g0fNbPMx2wyXVfSOEDjEQ@mail.gmail.com> (raw)
In-Reply-To: <ZXUMHlzfBp12Z4bY@mariano>

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).
>
> [...]
>
_______________________________________________
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".