[FFmpeg-devel] [PATCH v1 2/6] avfilter/af_silencedetect: document metadata

[Limin Wang]

On Tue, Oct 01, 2019 at 09:40:45AM +0200, Moritz Barsnick wrote:
> On Mon, Sep 30, 2019 at 21:36:43 +0800, lance.lmwang at gmail.com wrote:
> > -The printed times and duration are expressed in seconds.
> > +The printed times and duration are expressed in seconds. The @code{lavfi.silence_start}
> > +or @code{lavfi.silence_start.X} metadata key is set on the first frame whose timestamp
> > +equals or exceeds the detection duration and it contains the timestamp of the first
> > +frame of the silence.
> > +
> > +The @code{lavfi.silence_duration} or @code{lavfi.silence_duration.X} and
> > + at code{lavfi.silence_end} or @code{lavfi.silence_end.X}metadata keys are set on the
> > +first frame after the silence. Where @code{X} is the channel number and .X is used
> > +if @option{mono} is enabled.
> 
> I don't understand. Literal ".X" (which you should write as
> "@code{.X}", by the way)? And if not mono, it's ".0", ".1"? And when is
> ".N" completely omitted?
By the code,  if it's mono mode, it'll output lavfi.silence_duration metadata, if not mono,
it'll output lavfi.silence_duration.0, lavfi.silence_duration.1(0, 1 is the channel number).
Please advise how to describe this clearly.
> 
> This is a bit unclear to me, without experimenting or looking at the
> code.
> 
> Cheers,
> Moritz
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".