[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