diff mbox

[FFmpeg-devel] add locale month names to av_small_strptime

Message ID 20170221000502.GY5776@nb4
State Not Applicable
Headers show

Commit Message

Michael Niedermayer Feb. 21, 2017, 12:05 a.m. UTC 
On Mon, Feb 20, 2017 at 03:30:43PM -0500, Micah Galizia wrote:
> Signed-off-by: Micah Galizia <micahgalizia@gmail.com>
> ---
>  libavutil/parseutils.c       | 28 ++++++++++++++++++++++++++++
>  libavutil/tests/parseutils.c |  7 +++++++
>  2 files changed, 35 insertions(+)

breaks fate
Test parseutils failed. Look at tests/data/fate/parseutils.err for details.


[...]

Comments

Micah Galizia Feb. 21, 2017, 12:48 a.m. UTC | #1 
Sorry about that -- this actually makes sense now -- I thought it was a bit odd that the test program had no asserts or anything to validate the output...

I've addressed the fate issues in this patch -- thanks again for the review.
diff mbox

Patch

--- ./tests/ref/fate/parseutils 2017-02-20 17:22:51.749108272 +0100
+++ tests/data/fate/parseutils  2017-02-21 01:04:00.337691174 +0100
@@ -68,6 +68,13 @@ 
 fmt:'%Y - %m - %d' spec:'2012-12-21' -> 2012-12-21 00:00:00
 fmt:'%Y-%m-%d %H:%M:%S' spec:'2012-12-21 20:12:21' -> 2012-12-21 20:12:21
 fmt:'  %Y - %m - %d %H : %M : %S' spec:'   2012 - 12 -  21   20 : 12 : 21' -> 2012-12-21 20:12:21
+fmt:'  %Y - %b - %d %H : %M : %S' spec:'   2012 - nOV -  21   20 : 12 : 21' -> 2012-11-21 20:12:21
+fmt:'  %Y - %B - %d %H : %M : %S' spec:'   2012 - nOVemBeR -  21   20 : 12 : 21' -> 2012-11-21 20:12:21
+fmt:'  %Y - %B%d %H : %M : %S' spec:'   2012 - may21   20 : 12 : 21' -> 2012-05-21 20:12:21
+fmt:'  %Y - %B%d %H : %M : %S' spec:'   2012 - mby21   20 : 12 : 21' -> error
+fmt:'  %Y - %B - %d %H : %M : %S' spec:'   2012 - JunE -  21   20 : 12 : 21' -> 2012-06-21 20:12:21
+fmt:'  %Y - %B - %d %H : %M : %S' spec:'   2012 - Jane -  21   20 : 12 : 21' -> error
+fmt:'  %Y - %B - %d %H : %M : %S' spec:'   2012 - January -  21   20 : 12 : 21' -> 2012-01-21 20:12:21

 Testing av_parse_time()
 (now is 2012-03-17 09:14:13.2 +0100, local time is UTC+1)