[FFmpeg-cvslog] avformat/mxfdec: replace obsolete comment

Marton Balint git at videolan.org
Tue Dec 25 21:21:41 EET 2018


ffmpeg | branch: master | Marton Balint <cus at passwd.hu> | Sat Dec 22 19:25:13 2018 +0100| [d0e43cf1a89c03913110c1b8acb6dd5acfc8693a] | committer: Marton Balint

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 <cus at passwd.hu>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d0e43cf1a89c03913110c1b8acb6dd5acfc8693a
---

 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);



More information about the ffmpeg-cvslog mailing list