From d0e43cf1a89c03913110c1b8acb6dd5acfc8693a Mon Sep 17 00:00:00 2001 From: Marton Balint Date: Sat, 22 Dec 2018 19:25:13 +0100 Subject: [PATCH] avformat/mxfdec: replace obsolete comment We no longer use strftime directly but use av_timegm to get an int64_t timestamp. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 6e96107498..d78f8ad2e4 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -2563,9 +2563,7 @@ static int64_t mxf_timestamp_to_int64(uint64_t timestamp) time.tm_min = (timestamp >> 16 & 0xFF); time.tm_sec = (timestamp >> 8 & 0xFF); - /* msvcrt versions of strftime calls the invalid parameter handler - * (aborting the process if one isn't set) if the parameters are out - * of range. */ + /* Clip values for legacy reasons. Maybe we should return error instead? */ time.tm_mon = av_clip(time.tm_mon, 0, 11); time.tm_mday = av_clip(time.tm_mday, 1, 31); time.tm_hour = av_clip(time.tm_hour, 0, 23);