Browse Source

We should be using a string constant for the strftime format string

Sam Lantinga 9 years ago
parent
commit
9dfe54006e
1 changed files with 1 additions and 5 deletions
  1. 1 5
      src/test/SDL_test_log.c

+ 1 - 5
src/test/SDL_test_log.c

@@ -50,23 +50,19 @@
  *
  * \return Ascii representation of the timestamp in localtime in the format '08/23/01 14:55:02'
  */
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wformat-nonliteral"
 char *SDLTest_TimestampToString(const time_t timestamp)
 {
     time_t copy;
     static char buffer[64];
     struct tm *local;
-    const char *fmt = "%x %X";
 
     SDL_memset(buffer, 0, sizeof(buffer));
     copy = timestamp;
     local = localtime(&copy);
-    strftime(buffer, sizeof(buffer), fmt, local);
+    strftime(buffer, sizeof(buffer), "%x %X", local);
 
     return buffer;
 }
-#pragma GCC diagnostic pop
 
 /*
  * Prints given message with a timestamp in the TEST category and INFO priority.