[FFmpeg-devel] [PATCH] Remove localtime_r from os_support
Mon Jun 25 01:20:27 CEST 2007
Ramiro Ribeiro Polla <ramiro at lisha.ufsc.br> writes:
> It seems FFmpeg checks for it in configure, provides an implementation
> if it doesn't exist, but this function isn't used anywhere.
> grep for localtime_r shows:
> configure: localtime_r
> configure:check_func localtime_r
> libavformat/os_support.c:struct tm *localtime_r(const time_t *t,
> struct tm *tp)
> It seems it was added in 2003 for MinGW, but I couldn't find the exact
> revision. Was it used a long time ago? Is it safe to remove this now?
It's been in there much longer than that, for what reason I cannot
tell. The last use of it was apparently (thanks git) removed in
By all means apply the patch.
mans at mansr.com
More information about the ffmpeg-devel