[FFmpeg-devel] [PATCH] parseutils: accept only full "ms" and "us" prefixes

2018-03-03 Thread Rostislav Pehlivanov
The commit which added those was pushed prematurely before anyone could object to illogical suffixes like just m for milliseconds. Without this, we'd be locked into never being able to implement the "m" suffix for minutes. Signed-off-by: Rostislav Pehlivanov --- libavutil/parseutils.c | 10 -

Re: [FFmpeg-devel] [PATCH] parseutils: accept only full "ms" and "us" prefixes

2018-03-03 Thread Bodecs Bela
2018.03.03. 21:19 keltezéssel, Rostislav Pehlivanov írta: The commit which added those was pushed prematurely before anyone could object to illogical suffixes like just m for milliseconds. Without this, we'd be locked into never being able to implement the "m" suffix for minutes. Signed-off-by

Re: [FFmpeg-devel] [PATCH] parseutils: accept only full "ms" and "us" prefixes

2018-03-04 Thread Rostislav Pehlivanov
On 3 March 2018 at 20:19, Rostislav Pehlivanov wrote: > The commit which added those was pushed prematurely before anyone could > object > to illogical suffixes like just m for milliseconds. Without this, we'd be > locked > into never being able to implement the "m" suffix for minutes. > > Signed

Re: [FFmpeg-devel] [PATCH] parseutils: accept only full "ms" and "us" prefixes

2018-03-05 Thread Aurelien Jacobs
On Sat, Mar 03, 2018 at 08:19:43PM +, Rostislav Pehlivanov wrote: > The commit which added those was pushed prematurely before anyone could object > to illogical suffixes like just m for milliseconds. What you call illogical is following the same convention as pretty much all numeric parameter

Re: [FFmpeg-devel] [PATCH] parseutils: accept only full "ms" and "us" prefixes

2018-03-05 Thread Rostislav Pehlivanov
On 5 March 2018 at 16:47, Aurelien Jacobs wrote: > On Sat, Mar 03, 2018 at 08:19:43PM +, Rostislav Pehlivanov wrote: > > The commit which added those was pushed prematurely before anyone could > object > > to illogical suffixes like just m for milliseconds. > > What you call illogical is foll