[FFmpeg-devel] [PATCH] avutil/parseutils: only accept full us duration, do not accept mss duration
Marton Balint
cus at passwd.hu
Thu Mar 8 00:45:03 EET 2018
On Wed, 7 Mar 2018, Aurelien Jacobs wrote:
> On Tue, Mar 06, 2018 at 01:02:48AM +0100, Marton Balint wrote:
>> Accepting 'u' suffix for a time specification is neither intuitive nor
>> consistent (now that we don't accept m).
>
> The 'm' SI prefix is still accepted in various time options, and the 'u'
> prefix is still accepted in those options even after your patch, so you
> can't really argue that this patch improve consistency.
> (eg. -black_min_duration 5ms is still accepted).
> So this will surprise nobody that I don't like this patch.
This really is a cursed topic, I am not sure I follow, after the patch:
5ms is accepted
5us is accepted
5m is not accepted
5u is not accepted
You really insist on accepting '5u'? If not, then I can push the patch as
is, right?
Thanks,
Marton
More information about the ffmpeg-devel
mailing list