Ticket #134: mc-4.6.1-timefmt.diff

edit/edit.c

@@ -29 +29 @@
 #include "../src/cmd.h"         /* view_other_cmd() */
 #include "../src/user.h"        /* user_menu_cmd() */
 #include "../src/wtools.h"      /* query_dialog() */
+#include "../src/timefmt.h"     /* time formatting */
 
 /*
    what editor are we going to emulate? one of EDIT_KEY_EMULATION_NORMAL
@@ -2512 +2513 @@
         break;
 
     case CK_Date:{
-            time_t t;
-#ifdef HAVE_STRFTIME
             char s[1024];
             /* fool gcc to prevent a Y2K warning */
             char time_format[] = "_c";
             time_format[0] = '%';
-#endif
-            time (&t);
-#ifdef HAVE_STRFTIME
-            strftime (s, sizeof (s), time_format, localtime (&t));
+
+            FMT_LOCALTIME_CURRENT(s, sizeof(s), time_format);
             edit_print_string (edit, s);
-#else
-            edit_print_string (edit, ctime (&t));
-#endif
             edit->force |= REDRAW_PAGE;
             break;
         }

src/Makefile.am

@@ -57 +57 @@
         popt.c poptconfig.c popt.h popthelp.c poptint.h poptparse.c     \
         profile.c profile.h regex.c rxvt.c screen.c setup.c setup.h     \
         slint.c subshell.c subshell.h textconf.c textconf.h             \
-        tree.c tree.h treestore.c treestore.h tty.h user.c user.h       \
-        util.c util.h utilunix.c view.c view.h vfsdummy.h widget.c      \
-        widget.h win.c win.h wtools.c wtools.h                          \
+        tree.c tree.h treestore.c treestore.h timefmt.h tty.h user.c    \
+        user.h util.c util.h utilunix.c view.c view.h vfsdummy.h        \
+        widget.c widget.h win.c win.h wtools.c wtools.h                 \
         x11conn.h x11conn.c
 
 if CHARSET

src/timefmt.h

@@ +1 @@
+#ifndef __UTIL_TIMEFMT_H
+#define __UTIL_TIMEFMT_H
+
+#include <sys/types.h>
+
+#define INVALID_TIME_TEXT       "(invalid)"
+
+#ifdef HAVE_STRFTIME
+
+/* safe localtime formatting - strftime()-using version */
+#define FMT_LOCALTIME(buffer, bufsize, fmt, when)                       \
+    {                                                                   \
+        struct tm *whentm;                                              \
+        whentm = localtime(&when);                                      \
+        if (whentm == NULL)                                             \
+        {                                                               \
+            strncpy(buffer, INVALID_TIME_TEXT, bufsize);                \
+            buffer[bufsize-1] = 0;                                      \
+        }                                                               \
+        else                                                            \
+        {                                                               \
+            strftime(buffer, bufsize, fmt, whentm);                     \
+        }                                                               \
+    }                                                                   \
+
+#else
+
+/* fallback when strftime/localtime not available */
+#define FMT_LOCALTIME(buffer,bufsize,fmt,when)                  \
+    {                                                           \
+        ctime_r(when,buffer);                                   \
+    }                                                           \
+
+#endif
+
+#define FMT_LOCALTIME_CURRENT(buffer, bufsize, fmt)             \
+    {                                                           \
+        time_t __current_time;                                  \
+        time(&__current_time);                                  \
+        FMT_LOCALTIME(buffer,bufsize,fmt,__current_time);       \
+    }
+
+#endif                          /* !__UTIL_H */

src/util.c

@@ -39 +39 @@
 #include "cmd.h"                /* guess_message_value */
 #include "mountlist.h"
 #include "win.h"                /* xterm_flag */
+#include "timefmt.h"
 
 #ifdef HAVE_CHARSET
 #include "charsets.h"
@@ -694 +695 @@
 short-month-name sizes for different locales */ 
 size_t i18n_checktimelength (void)
 {
-    size_t length, a, b;
-    char buf [MAX_I18NTIMELENGTH + 1];
     time_t testtime = time (NULL);
-    
-    a = strftime (buf, sizeof(buf)-1, _("%b %e %H:%M"), localtime(&testtime));
-    b = strftime (buf, sizeof(buf)-1, _("%b %e  %Y"), localtime(&testtime));
-    
-    length = max (a, b);
-    
+    struct tm* lt = localtime(&testtime);
+    size_t length;
+
+    if (lt == NULL)
+    {
+        // huh, localtime() doesnt seem to work ... falling back to "(invalid)"
+        length = strlen(INVALID_TIME_TEXT);
+    }
+    else
+    {
+        char buf [MAX_I18NTIMELENGTH + 1];
+        size_t a, b;
+        a = strftime (buf, sizeof(buf)-1, _("%b %e %H:%M"), lt);
+        b = strftime (buf, sizeof(buf)-1, _("%b %e  %Y"), lt);
+        length = max (a, b);
+    }
+
     /* Don't handle big differences. Use standard value (email bug, please) */
     if ( length > MAX_I18NTIMELENGTH || length < MIN_I18NTIMELENGTH )
         length = STD_I18NTIMELENGTH;
-    
+
     return length;
 }
 
@@ -740 +750 @@
     else
         fmt = fmttime;
     
-    strftime (timebuf, i18n_timelength, fmt, localtime(&when));
+    FMT_LOCALTIME(timebuf, i18n_timelength, fmt, when);
     return timebuf;
 }