[FFmpeg-devel] [PATCH] parseutils: add support for ms and us suffix for AV_OPT_TYPE_DURATION
Marton Balint
cus at passwd.hu
Sat Mar 3 00:59:33 EET 2018
On Fri, 2 Mar 2018, Hendrik Leppkes wrote:
> On Fri, Mar 2, 2018 at 11:19 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
>> On Fri, Mar 02, 2018 at 10:02:58PM +0000, Rostislav Pehlivanov wrote:
>>> On 2 March 2018 at 21:57, Aurelien Jacobs <aurel at gnuage.org> wrote:
>>>
>>> > On Fri, Mar 02, 2018 at 09:39:48PM +0100, Michael Niedermayer wrote:
>>> > > On Thu, Mar 01, 2018 at 09:41:20PM +0100, Aurelien Jacobs wrote:
>>> > > > supported suffixes are:
>>> > > > - s: seconds (default when no suffix specified)
>>> > > > - m or ms: milliseconds
>>> > > > - u or us: microseconds
I don't see much benefit in accepting the SI prefixes without actual unit,
the purpose of this whoule patch is intuitive and readable command line,
and an SI prefix alone, without a unit is not intuitive. So I'd drop the
SI-prefix-only variants.
Thanks,
Marton
More information about the ffmpeg-devel
mailing list