g_date_time_format: drop %N format
authorRyan Lortie <desrt@desrt.ca>
Fri, 2 Sep 2011 17:38:03 +0000 (13:38 -0400)
committerRyan Lortie <desrt@desrt.ca>
Fri, 2 Sep 2011 22:10:38 +0000 (18:10 -0400)
%N is not specified in any standards document, but we use it to display
the number of microseconds.

The fact that our our current implementation of it is nearly useless
(since it does not zero-pad) coupled with the high chance that a future
version of the C standard may specify it with another meaning means that
we should drop it.

https://bugzilla.gnome.org/show_bug.cgi?id=658061

glib/gdatetime.c

index eeca45f..fede9f4 100644 (file)
@@ -2222,11 +2222,6 @@ get_numeric_format (gchar    *fmt,
  *    the minute as a decimal number (range 00 to 59)
  *  </simpara></listitem></varlistentry>
  *  <varlistentry><term>
- *    <literal>%%N</literal>:
- *   </term><listitem><simpara>
- *    the micro-seconds as a decimal number
- *  </simpara></listitem></varlistentry>
- *  <varlistentry><term>
  *    <literal>%%p</literal>:
  *   </term><listitem><simpara>
  *    either "AM" or "PM" according to the given time value, or the
@@ -2490,9 +2485,6 @@ g_date_time_format (GDateTime   *datetime,
                   get_numeric_format (fmt, sizeof(fmt), alt_digits, pad_set ? pad : '0', 2);
                   g_string_append_printf (outstr, fmt, g_date_time_get_minute (datetime));
                   break;
-                case 'N':
-                  g_string_append_printf (outstr, "%"G_GUINT64_FORMAT, datetime->usec % USEC_PER_SECOND);
-                  break;
                 case 'O':
                   alt_digits = TRUE;
                   goto next_mod;