Re: [FFmpeg-devel] [PATCH v2] ffprobe: add -codec:<media_spec> option

[Stefano Sabatini <stefasab@gmail.com>]

On date Thursday 2025-07-31 13:54:52 +0200, ffmpeg-devel Mailing List wrote:
> On Thu, Jul 31, 2025 at 12:01 AM Stefano Sabatini <stefasab@gmail.com> wrote:
> >
> > On date Friday 2025-06-27 10:23:56 +0200, ffmpeg-devel Mailing List wrote:
> > > From: Maryla Ustarroz-Calonge <maryla@google.com>
> > > Subject: [PATCH v2] ffprobe: add -codec:<media_spec> option
> > > Date: Fri, 27 Jun 2025 10:23:56 +0200
> > > To: ffmpeg-devel@ffmpeg.org
> > > X-Mailer: git-send-email 2.50.0.727.gbf7dc18ff4-goog
> > >
> > > And -c:<media_spec> variant.
> > > Update ffprobe.texi.
> > > opt_codec() is mostly copied over from ffplay.c
> > >
> > > Signed-off-by: Maryla Ustarroz-Calonge <maryla@google.com>
> > > ---
> > >  Changelog         |   2 +-
> > >  doc/ffprobe.texi  |   8 ++++
> > >  fftools/ffprobe.c | 117 +++++++++++++++++++++++++++++++++++++++++-----
> > >  3 files changed, 114 insertions(+), 13 deletions(-)
> > >
> > > diff --git a/Changelog b/Changelog
> > > index 4217449438..ae73611222 100644
> > > --- a/Changelog
> > > +++ b/Changelog
> > > @@ -18,7 +18,7 @@ version <next>:
> > >  - APV encoding support through a libopenapv wrapper
> > >  - VVC decoder supports all content of SCC (Screen Content Coding):
> > >    IBC (Inter Block Copy), Palette Mode and ACT (Adaptive Color Transform
> > > -
> > > +- ffprobe -codec option
> > >
> > >  version 7.1:
> > >  - Raw Captions with Time (RCWT) closed caption demuxer
> > > diff --git a/doc/ffprobe.texi b/doc/ffprobe.texi
> > > index 8834df8d35..4ce0c8b583 100644
> > > --- a/doc/ffprobe.texi
> > > +++ b/doc/ffprobe.texi
> > > @@ -368,6 +368,14 @@ Read @var{input_url}.
> > >  Write output to @var{output_url}. If not specified, the output is sent
> > >  to stdout.
> > >
> > > +@item -c:@var{media_specifier} @var{codec_name}
> > > +Force a specific decoder implementation for the stream identified by
> > > +@var{media_specifier}, which can assume the values @code{a} (audio),
> > > +@code{v} (video), @code{s} (subtitle), and @code{d} (data).
> > > +
> > > +@item -codec:@var{media_specifier} @var{codec_name}
> > > +Alias for -c:@var{media_specifier}.
> >
> > Nit: I'd put the long version before the short one, but keep as is if
> > this is consistent with the other docs.
> 
> I just sent a v3 addressing your comments.
> In the texi file, I merged the two entries for -c and -codec with -c first,
> as is done in ffmpeg.texi
> Note that in the flag descriptions in ffprobe.c, -codec is marked as being
> an alias of -c, which seems backwards to me, but I did it this way to be
> consistent with ffmpeg.c

I just pushed version 3, thanks!
_______________________________________________
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".