[gnome-shell] src: Stop using g_get_current_time()



commit 9b7f228f8ef224c36fd8809942ec78ce071fdbd7
Author: Florian Müllner <fmuellner gnome org>
Date:   Wed Jul 31 20:29:46 2019 +0200

    src: Stop using g_get_current_time()
    
    It isn't 2k38 safe and has therefore been deprecated. Replace it
    with GDateTime or g_get_real_time() as appropriate.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/657

 src/calendar-server/gnome-shell-calendar-server.c | 18 +++++++-----------
 src/hotplug-sniffer/hotplug-sniffer.c             | 18 +++++++-----------
 src/shell-app-usage.c                             |  4 +---
 src/shell-polkit-authentication-agent.c           | 18 +++++++-----------
 4 files changed, 22 insertions(+), 36 deletions(-)
---
diff --git a/src/calendar-server/gnome-shell-calendar-server.c 
b/src/calendar-server/gnome-shell-calendar-server.c
index d5b6e01f7b..2da4d113ae 100644
--- a/src/calendar-server/gnome-shell-calendar-server.c
+++ b/src/calendar-server/gnome-shell-calendar-server.c
@@ -1102,12 +1102,10 @@ main (int    argc,
 static void __attribute__((format(printf, 1, 0)))
 print_debug (const gchar *format, ...)
 {
-  gchar *s;
+  g_autofree char *s = NULL;
+  g_autofree char *timestamp = NULL;
   va_list ap;
-  gchar timebuf[64];
-  GTimeVal now;
-  time_t now_t;
-  struct tm broken_down;
+  g_autoptr (GDateTime) now = NULL;
   static volatile gsize once_init_value = 0;
   static gboolean show_debug = FALSE;
   static guint pid = 0;
@@ -1122,17 +1120,15 @@ print_debug (const gchar *format, ...)
   if (!show_debug)
     goto out;
 
-  g_get_current_time (&now);
-  now_t = now.tv_sec;
-  localtime_r (&now_t, &broken_down);
-  strftime (timebuf, sizeof timebuf, "%H:%M:%S", &broken_down);
+  now = g_date_time_new_now_local ();
+  timestamp = g_date_time_format (now, "%H:%M:%S");
 
   va_start (ap, format);
   s = g_strdup_vprintf (format, ap);
   va_end (ap);
 
-  g_print ("gnome-shell-calendar-server[%d]: %s.%03d: %s\n", pid, timebuf, (gint) (now.tv_usec / 1000), s);
-  g_free (s);
+  g_print ("gnome-shell-calendar-server[%d]: %s.%03d: %s\n",
+           pid, timestamp, g_date_time_get_microsecond (now), s);
  out:
   ;
 }
diff --git a/src/hotplug-sniffer/hotplug-sniffer.c b/src/hotplug-sniffer/hotplug-sniffer.c
index 31e45795ed..b8d7fa0ea8 100644
--- a/src/hotplug-sniffer/hotplug-sniffer.c
+++ b/src/hotplug-sniffer/hotplug-sniffer.c
@@ -269,12 +269,10 @@ main (int    argc,
 static void __attribute__((format(printf, 1, 0)))
 print_debug (const gchar *format, ...)
 {
-  gchar *s;
+  g_autofree char *s = NULL;
+  g_autofree char *timestamp = NULL;
   va_list ap;
-  gchar timebuf[64];
-  GTimeVal now;
-  time_t now_t;
-  struct tm broken_down;
+  g_autoptr (GDateTime) now = NULL;
   static volatile gsize once_init_value = 0;
   static gboolean show_debug = FALSE;
   static guint pid = 0;
@@ -289,17 +287,15 @@ print_debug (const gchar *format, ...)
   if (!show_debug)
     goto out;
 
-  g_get_current_time (&now);
-  now_t = now.tv_sec;
-  localtime_r (&now_t, &broken_down);
-  strftime (timebuf, sizeof timebuf, "%H:%M:%S", &broken_down);
+  now = g_date_time_new_now_local ();
+  timestamp = g_date_time_format (now, "%H:%M:%S");
 
   va_start (ap, format);
   s = g_strdup_vprintf (format, ap);
   va_end (ap);
 
-  g_print ("gnome-shell-hotplug-sniffer[%d]: %s.%03d: %s\n", pid, timebuf, (gint) (now.tv_usec / 1000), s);
-  g_free (s);
+  g_print ("gnome-shell-hotplug-sniffer[%d]: %s.%03d: %s\n",
+           pid, timestamp, g_date_time_get_microsecond (now), s);
  out:
   ;
 }
diff --git a/src/shell-app-usage.c b/src/shell-app-usage.c
index b8dc768cae..b8c3e75674 100644
--- a/src/shell-app-usage.c
+++ b/src/shell-app-usage.c
@@ -123,9 +123,7 @@ static void on_enable_monitoring_key_changed (GSettings     *settings,
 static long
 get_time (void)
 {
-  GTimeVal tv;
-  g_get_current_time (&tv);
-  return tv.tv_sec;
+  return g_get_real_time () / G_TIME_SPAN_SECOND;
 }
 
 static void
diff --git a/src/shell-polkit-authentication-agent.c b/src/shell-polkit-authentication-agent.c
index bca967fa1b..6815f01aff 100644
--- a/src/shell-polkit-authentication-agent.c
+++ b/src/shell-polkit-authentication-agent.c
@@ -21,24 +21,20 @@
 static void
 print_debug (const gchar *format, ...)
 {
-  gchar *s;
+  g_autofree char *s = NULL;
+  g_autofree char *timestamp = NULL;
+  g_autoptr (GDateTime) now = NULL;
   va_list ap;
-  gchar timebuf[64];
-  GTimeVal now;
-  time_t now_t;
-  struct tm broken_down;
 
-  g_get_current_time (&now);
-  now_t = now.tv_sec;
-  localtime_r (&now_t, &broken_down);
-  strftime (timebuf, sizeof timebuf, "%H:%M:%S", &broken_down);
+  now = g_date_time_new_now_local ();
+  timestamp = g_date_time_format (now, "%H:%M:%S");
 
   va_start (ap, format);
   s = g_strdup_vprintf (format, ap);
   va_end (ap);
 
-  g_print ("ShellPolkitAuthenticationAgent: %s.%03d: %s\n", timebuf, (gint) (now.tv_usec / 1000), s);
-  g_free (s);
+  g_print ("ShellPolkitAuthenticationAgent: %s.%03d: %s\n",
+           timestamp, g_date_time_get_microsecond (now), s);
 }
 #else
 static void


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]