Include config.h to get HAVE_LOCALTIME_R macro.
[platform/upstream/glib.git] / gdate.c
diff --git a/gdate.c b/gdate.c
index 6de541c..44a6bad 100644 (file)
--- a/gdate.c
+++ b/gdate.c
  * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
+/* 
+ * MT safe
+ */
+
+#ifdef HAVE_CONFIG_H
+#include <config.h>
+#endif
+
 #include "glib.h"
 
 #include <time.h>
@@ -33,15 +42,15 @@ g_date_new ()
 }
 
 GDate*
-g_date_new_mdy (GDateMonth m, GDateDay day, GDateYear y)
+g_date_new_dmy (GDateDay day, GDateMonth m, GDateYear y)
 {
   GDate *d;
-  g_return_val_if_fail (g_date_valid_mdy (m, day, y), NULL);
+  g_return_val_if_fail (g_date_valid_dmy (day, m, y), NULL);
   
   d = g_new (GDate, 1);
   
   d->julian = FALSE;
-  d->mdy    = TRUE;
+  d->dmy    = TRUE;
   
   d->month = m;
   d->day   = day;
@@ -61,7 +70,7 @@ g_date_new_julian (guint32 j)
   d = g_new (GDate, 1);
   
   d->julian = TRUE;
-  d->mdy    = FALSE;
+  d->dmy    = FALSE;
   
   d->julian_days = j;
   
@@ -78,29 +87,12 @@ g_date_free (GDate *d)
   g_free (d);
 }
 
-
-
 gboolean     
 g_date_valid (GDate       *d)
 {
   g_return_val_if_fail (d != NULL, FALSE);
   
-  if (d->julian && d->mdy) 
-    {
-      return ( g_date_valid_julian (d->julian_days) && 
-               g_date_valid_mdy (d->month, d->day, d->year) );
-      
-    }
-  else if (d->mdy)
-    {
-      return g_date_valid_mdy (d->month, d->day, d->year);
-    }
-  else if (d->julian)
-    {
-      return g_date_valid_julian (d->julian_days);
-    }
-  else 
-    return FALSE;
+  return (d->julian || d->dmy);
 }
 
 static const guint8 days_in_months[2][13] = 
@@ -146,8 +138,8 @@ g_date_valid_julian (guint32      j)
 }
 
 gboolean     
-g_date_valid_mdy (GDateMonth   m
-                 GDateDay     d
+g_date_valid_dmy (GDateDay     d
+                  GDateMonth   m
                  GDateYear    y)
 {
   return ( (m > G_DATE_BAD_MONTH) &&
@@ -169,9 +161,9 @@ g_date_update_julian (GDate *d)
   gint index;
   
   g_return_if_fail (d != NULL);
-  g_return_if_fail (d->mdy);
+  g_return_if_fail (d->dmy);
   g_return_if_fail (!d->julian);
-  g_return_if_fail (g_date_valid_mdy (d->month, d->day, d->year));
+  g_return_if_fail (g_date_valid_dmy (d->day, d->month, d->year));
   
   /* What we actually do is: multiply years * 365 days in the year,
    *  add the number of years divided by 4, subtract the number of
@@ -197,7 +189,7 @@ g_date_update_julian (GDate *d)
 }
 
 static void 
-g_date_update_mdy (GDate *d)
+g_date_update_dmy (GDate *d)
 {
   GDateYear y;
   GDateMonth m;
@@ -207,7 +199,7 @@ g_date_update_mdy (GDate *d)
   
   g_return_if_fail (d != NULL);
   g_return_if_fail (d->julian);
-  g_return_if_fail (!d->mdy);
+  g_return_if_fail (!d->dmy);
   g_return_if_fail (g_date_valid_julian (d->julian_days));
   
   /* Formula taken from the Calendar FAQ; the formula was for the
@@ -230,10 +222,10 @@ g_date_update_mdy (GDate *d)
   y = 100 * B + D - 4800 + (M/10);
   
 #ifdef G_ENABLE_DEBUG
-  if (!g_date_valid_mdy (m, day, y)) 
+  if (!g_date_valid_dmy (day, m, y)) 
     {
-      g_warning ("\nOOPS julian: %u  computed mdy: %u %u %u\n", 
-                d->julian_days, m, day, y);
+      g_warning ("\nOOPS julian: %u  computed dmy: %u %u %u\n", 
+                d->julian_days, day, m, y);
     }
 #endif
   
@@ -241,7 +233,7 @@ g_date_update_mdy (GDate *d)
   d->day   = day;
   d->year  = y;
   
-  d->mdy = TRUE;
+  d->dmy = TRUE;
 }
 
 GDateWeekday 
@@ -265,11 +257,11 @@ g_date_month (GDate *d)
   g_return_val_if_fail (d != NULL, G_DATE_BAD_MONTH);
   g_return_val_if_fail (g_date_valid (d), G_DATE_BAD_MONTH);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, G_DATE_BAD_MONTH);
+  g_return_val_if_fail (d->dmy, G_DATE_BAD_MONTH);
   
   return d->month;
 }
@@ -280,11 +272,11 @@ g_date_year (GDate *d)
   g_return_val_if_fail (d != NULL, G_DATE_BAD_YEAR);
   g_return_val_if_fail (g_date_valid (d), G_DATE_BAD_YEAR);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, G_DATE_BAD_YEAR);  
+  g_return_val_if_fail (d->dmy, G_DATE_BAD_YEAR);  
   
   return d->year;
 }
@@ -295,11 +287,11 @@ g_date_day (GDate *d)
   g_return_val_if_fail (d != NULL, G_DATE_BAD_DAY);
   g_return_val_if_fail (g_date_valid (d), G_DATE_BAD_DAY);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, G_DATE_BAD_DAY);  
+  g_return_val_if_fail (d->dmy, G_DATE_BAD_DAY);  
   
   return d->day;
 }
@@ -327,11 +319,11 @@ g_date_day_of_year (GDate *d)
   g_return_val_if_fail (d != NULL, 0);
   g_return_val_if_fail (g_date_valid (d), 0);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, 0);  
+  g_return_val_if_fail (d->dmy, 0);  
   
   index = g_date_is_leap_year (d->year) ? 1 : 0;
   
@@ -348,15 +340,15 @@ g_date_monday_week_of_year (GDate *d)
   g_return_val_if_fail (d != NULL, 0);
   g_return_val_if_fail (g_date_valid (d), 0);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, 0);  
+  g_return_val_if_fail (d->dmy, 0);  
   
   g_date_clear (&first, 1);
   
-  g_date_set_mdy (&first, 1, 1, d->year);
+  g_date_set_dmy (&first, 1, 1, d->year);
   
   wd = g_date_weekday (&first) - 1; /* make Monday day 0 */
   day = g_date_day_of_year (d) - 1;
@@ -374,15 +366,15 @@ g_date_sunday_week_of_year (GDate *d)
   g_return_val_if_fail (d != NULL, 0);
   g_return_val_if_fail (g_date_valid (d), 0);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, 0);  
+  g_return_val_if_fail (d->dmy, 0);  
   
   g_date_clear (&first, 1);
   
-  g_date_set_mdy (&first, 1, 1, d->year);
+  g_date_set_dmy (&first, 1, 1, d->year);
   
   wd = g_date_weekday (&first);
   if (wd == 7) wd = 0; /* make Sunday day 0 */
@@ -400,6 +392,8 @@ g_date_clear (GDate       *d, guint ndates)
   memset (d, 0x0, ndates*sizeof (GDate)); 
 }
 
+G_LOCK_DECLARE_STATIC (g_date_global);
+
 /* These are for the parser, output to the user should use *
  * g_date_strftime () - this creates more never-freed memory to annoy
  * all those memory debugger users. :-) 
@@ -419,9 +413,9 @@ static gchar *short_month_names[13] =
 static gchar *current_locale = NULL;
 
 /* order of these in the current locale */
-static GDateMDY mdy_order[3] = 
+static GDateDMY dmy_order[3] = 
 {
-  G_DATE_MONTH, G_DATE_DAY, G_DATE_YEAR
+   G_DATE_DAY, G_DATE_MONTH, G_DATE_YEAR
 };
 
 /* Where to chop two-digit years: i.e., for the 1930 default, numbers
@@ -446,6 +440,7 @@ typedef struct _GDateParseTokens GDateParseTokens;
 
 #define NUM_LEN 10
 
+/* HOLDS: g_date_global_lock */
 static void
 g_date_fill_parse_tokens (const gchar *str, GDateParseTokens *pt)
 {
@@ -505,7 +500,7 @@ g_date_fill_parse_tokens (const gchar *str, GDateParseTokens *pt)
               if (found != NULL)
                 {
                   pt->month = i;
-                  return;
+                 return;
                 }
             }
          
@@ -519,12 +514,13 @@ g_date_fill_parse_tokens (const gchar *str, GDateParseTokens *pt)
                   return;
                 }
             }
-         
+
           ++i;
-        }
+        }      
     }
 }
 
+/* HOLDS: g_date_global_lock */
 static void
 g_date_prepare_to_parse (const gchar *str, GDateParseTokens *pt)
 {
@@ -553,7 +549,7 @@ g_date_prepare_to_parse (const gchar *str, GDateParseTokens *pt)
       
       while (i < 13) 
         {
-          g_date_set_mdy (&d, i, 1, 1);
+          g_date_set_dmy (&d, 1, i, 1);
          
           g_return_if_fail (g_date_valid (&d));
          
@@ -572,9 +568,11 @@ g_date_prepare_to_parse (const gchar *str, GDateParseTokens *pt)
           ++i;
         }
       
-      /* Determine MDY order */
+      /* Determine DMY order */
       
-      g_date_set_mdy (&d, 7, 4, 1776); /* had to pick a random day */
+      /* had to pick a random day - don't change this, some strftimes
+       * are broken on some days, and this one is good so far. */
+      g_date_set_dmy (&d, 4, 7, 1976);
       
       g_date_strftime (buf, 127, "%x", &d);
       
@@ -586,15 +584,15 @@ g_date_prepare_to_parse (const gchar *str, GDateParseTokens *pt)
           switch (testpt.n[i])
             {
             case 7:
-              mdy_order[i] = G_DATE_MONTH;
+              dmy_order[i] = G_DATE_MONTH;
               break;
             case 4:
-              mdy_order[i] = G_DATE_DAY;
+              dmy_order[i] = G_DATE_DAY;
               break;
             case 76:
               using_twodigit_years = TRUE; /* FALL THRU */
-            case 1776:
-              mdy_order[i] = G_DATE_YEAR;
+            case 1976:
+              dmy_order[i] = G_DATE_YEAR;
               break;
             default:
               /* leave it unchanged */
@@ -620,7 +618,7 @@ g_date_prepare_to_parse (const gchar *str, GDateParseTokens *pt)
         i = 0;
         while (i < 3)
           {
-            switch (mdy_order[i])
+            switch (dmy_order[i])
               {
               case G_DATE_MONTH:
                 strings[i] = "Month";
@@ -658,6 +656,8 @@ g_date_set_parse (GDate       *d,
   /* set invalid */
   g_date_clear (d, 1);
   
+  G_LOCK (g_date_global);
+
   g_date_prepare_to_parse (str, &pt);
   
 #ifdef G_ENABLE_DEBUG
@@ -666,7 +666,11 @@ g_date_set_parse (GDate       *d,
 #endif
   
   
-  if (pt.num_ints == 4) return; /* presumably a typo; bail out. */
+  if (pt.num_ints == 4) 
+    {
+      G_UNLOCK (g_date_global);
+      return; /* presumably a typo; bail out. */
+    }
   
   if (pt.num_ints > 1)
     {
@@ -677,7 +681,7 @@ g_date_set_parse (GDate       *d,
       
       while (i < pt.num_ints && j < 3) 
         {
-          switch (mdy_order[j])
+          switch (dmy_order[j])
             {
             case G_DATE_MONTH:
            {
@@ -727,7 +731,7 @@ g_date_set_parse (GDate       *d,
         }
       
       
-      if (pt.num_ints == 3 && !g_date_valid_mdy (m, day, y))
+      if (pt.num_ints == 3 && !g_date_valid_dmy (day, m, y))
         {
           /* Try YYYY MM DD */
           y   = pt.n[0];
@@ -771,17 +775,18 @@ g_date_set_parse (GDate       *d,
   
   /* See if we got anything valid out of all this. */
   /* y < 8000 is to catch 19998 style typos; the library is OK up to 65535 or so */
-  if (y < 8000 && g_date_valid_mdy (m, day, y)) 
+  if (y < 8000 && g_date_valid_dmy (day, m, y)) 
     {
       d->month = m;
       d->day   = day;
       d->year  = y;
-      d->mdy   = TRUE;
+      d->dmy   = TRUE;
     }
 #ifdef G_ENABLE_DEBUG
   else 
-    g_message ("Rejected MDY %u %u %u", m, day, y);
+    g_message ("Rejected DMY %u %u %u", day, m, y);
 #endif
+  G_UNLOCK (g_date_global);
 }
 
 void         
@@ -789,28 +794,33 @@ g_date_set_time (GDate *d,
                 GTime  time)
 {
   time_t t = time;
-  struct tm *tm;
+  struct tm tm;
   
   g_return_if_fail (d != NULL);
   
-  tm = localtime (&t);
+#ifdef HAVE_LOCALTIME_R
+  localtime_r (&t, &tm);
+#else
+#  ifdef G_THREADS_ENABLED
+#  warning "the `g_date_set_time' function will not be MT-safe"
+#  warning "because there is no `localtime_r' on your system."
+#  endif
+  {
+    struct tm *ptm = localtime (&t);
+    g_assert (ptm);
+    memcpy ((void *) &tm, (void *) ptm, sizeof(struct tm));
+  }
+#endif
   
-  if (tm) 
-    {
-      d->julian = FALSE;
-      
-      d->month = tm->tm_mon + 1;
-      d->day   = tm->tm_mday;
-      d->year  = tm->tm_year + 1900;
-      
-      g_return_if_fail (g_date_valid_mdy (d->month, d->day, d->year));
-      
-      d->mdy    = TRUE;
-    }
-  else 
-    {
-      g_date_clear (d, 1);
-    }
+  d->julian = FALSE;
+  
+  d->month = tm.tm_mon + 1;
+  d->day   = tm.tm_mday;
+  d->year  = tm.tm_year + 1900;
+  
+  g_return_if_fail (g_date_valid_dmy (d->day, d->month, d->year));
+  
+  d->dmy    = TRUE;
 }
 
 void         
@@ -819,15 +829,16 @@ g_date_set_month (GDate     *d,
 {
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid_month (m));
-  
+
+  if (d->julian && !d->dmy) g_date_update_dmy(d);
   d->julian = FALSE;
   
   d->month = m;
   
-  if (g_date_valid_mdy (d->month, d->day, d->year))
-    d->mdy = TRUE;
+  if (g_date_valid_dmy (d->day, d->month, d->year))
+    d->dmy = TRUE;
   else 
-    d->mdy = FALSE;
+    d->dmy = FALSE;
 }
 
 void         
@@ -837,14 +848,15 @@ g_date_set_day (GDate     *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid_day (day));
   
+  if (d->julian && !d->dmy) g_date_update_dmy(d);
   d->julian = FALSE;
   
   d->day = day;
   
-  if (g_date_valid_mdy (d->month, d->day, d->year))
-    d->mdy = TRUE;
+  if (g_date_valid_dmy (d->day, d->month, d->year))
+    d->dmy = TRUE;
   else 
-    d->mdy = FALSE;
+    d->dmy = FALSE;
 }
 
 void         
@@ -854,24 +866,25 @@ g_date_set_year (GDate     *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid_year (y));
   
+  if (d->julian && !d->dmy) g_date_update_dmy(d);
   d->julian = FALSE;
   
   d->year = y;
   
-  if (g_date_valid_mdy (d->month, d->day, d->year))
-    d->mdy = TRUE;
+  if (g_date_valid_dmy (d->day, d->month, d->year))
+    d->dmy = TRUE;
   else 
-    d->mdy = FALSE;
+    d->dmy = FALSE;
 }
 
 void         
-g_date_set_mdy (GDate     *d, 
-                GDateMonth m, 
+g_date_set_dmy (GDate     *d, 
                 GDateDay   day, 
+                GDateMonth m, 
                 GDateYear  y)
 {
   g_return_if_fail (d != NULL);
-  g_return_if_fail (g_date_valid_mdy (m, day, y));
+  g_return_if_fail (g_date_valid_dmy (day, m, y));
   
   d->julian = FALSE;
   
@@ -879,7 +892,7 @@ g_date_set_mdy (GDate     *d,
   d->day   = day;
   d->year  = y;
   
-  d->mdy = TRUE;
+  d->dmy = TRUE;
 }
 
 void         
@@ -890,7 +903,7 @@ g_date_set_julian (GDate *d, guint32 j)
   
   d->julian_days = j;
   d->julian = TRUE;
-  d->mdy = FALSE;
+  d->dmy = FALSE;
 }
 
 
@@ -900,11 +913,11 @@ g_date_is_first_of_month (GDate *d)
   g_return_val_if_fail (d != NULL, FALSE);
   g_return_val_if_fail (g_date_valid (d), FALSE);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, FALSE);  
+  g_return_val_if_fail (d->dmy, FALSE);  
   
   if (d->day == 1) return TRUE;
   else return FALSE;
@@ -918,11 +931,11 @@ g_date_is_last_of_month (GDate *d)
   g_return_val_if_fail (d != NULL, FALSE);
   g_return_val_if_fail (g_date_valid (d), FALSE);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_val_if_fail (d->mdy, FALSE);  
+  g_return_val_if_fail (d->dmy, FALSE);  
   
   index = g_date_is_leap_year (d->year) ? 1 : 0;
   
@@ -943,7 +956,7 @@ g_date_add_days (GDate *d, guint ndays)
   g_return_if_fail (d->julian);
   
   d->julian_days += ndays;
-  d->mdy = FALSE;
+  d->dmy = FALSE;
 }
 
 void         
@@ -960,7 +973,7 @@ g_date_subtract_days (GDate *d, guint ndays)
   g_return_if_fail (d->julian_days > ndays);
   
   d->julian_days -= ndays;
-  d->mdy = FALSE;
+  d->dmy = FALSE;
 }
 
 void         
@@ -973,11 +986,11 @@ g_date_add_months (GDate       *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid (d));
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_if_fail (d->mdy);  
+  g_return_if_fail (d->dmy);  
   
   nmonths += d->month - 1;
   
@@ -1007,11 +1020,11 @@ g_date_subtract_months (GDate       *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid (d));
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_if_fail (d->mdy);  
+  g_return_if_fail (d->dmy);  
   
   years  = nmonths/12;
   months = nmonths%12;
@@ -1045,11 +1058,11 @@ g_date_add_years (GDate       *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid (d));
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_if_fail (d->mdy);  
+  g_return_if_fail (d->dmy);  
   
   d->year += nyears;
   
@@ -1071,11 +1084,11 @@ g_date_subtract_years (GDate       *d,
   g_return_if_fail (d != NULL);
   g_return_if_fail (g_date_valid (d));
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_if_fail (d->mdy);  
+  g_return_if_fail (d->dmy);  
   g_return_if_fail (d->year > nyears);
   
   d->year -= nyears;
@@ -1123,15 +1136,15 @@ g_date_monday_weeks_in_year (GDateYear  year)
   g_return_val_if_fail (g_date_valid_year (year), 0);
   
   g_date_clear (&d, 1);
-  g_date_set_mdy (&d, 1, 1, year);
+  g_date_set_dmy (&d, 1, 1, year);
   if (g_date_weekday (&d) == G_DATE_MONDAY) return 53;
-  g_date_set_mdy (&d, 12, 31, year);
+  g_date_set_dmy (&d, 31, 12, year);
   if (g_date_weekday (&d) == G_DATE_MONDAY) return 53;
   if (g_date_is_leap_year (year)) 
     {
-      g_date_set_mdy (&d, 1, 2, year);
+      g_date_set_dmy (&d, 2, 1, year);
       if (g_date_weekday (&d) == G_DATE_MONDAY) return 53;
-      g_date_set_mdy (&d, 12, 30, year);
+      g_date_set_dmy (&d, 30, 12, year);
       if (g_date_weekday (&d) == G_DATE_MONDAY) return 53;
     }
   return 52;
@@ -1145,15 +1158,15 @@ g_date_sunday_weeks_in_year (GDateYear  year)
   g_return_val_if_fail (g_date_valid_year (year), 0);
   
   g_date_clear (&d, 1);
-  g_date_set_mdy (&d, 1, 1, year);
+  g_date_set_dmy (&d, 1, 1, year);
   if (g_date_weekday (&d) == G_DATE_SUNDAY) return 53;
-  g_date_set_mdy (&d, 12, 31, year);
+  g_date_set_dmy (&d, 31, 12, year);
   if (g_date_weekday (&d) == G_DATE_SUNDAY) return 53;
   if (g_date_is_leap_year (year)) 
     {
-      g_date_set_mdy (&d, 1, 2, year);
+      g_date_set_dmy (&d, 2, 1, year);
       if (g_date_weekday (&d) == G_DATE_SUNDAY) return 53;
-      g_date_set_mdy (&d, 12, 30, year);
+      g_date_set_dmy (&d, 30, 12, year);
       if (g_date_weekday (&d) == G_DATE_SUNDAY) return 53;
     }
   return 52;
@@ -1179,7 +1192,7 @@ g_date_compare (GDate     *lhs,
           else if (lhs->julian_days > rhs->julian_days) return 1;
           else                                          return 0;
         }
-      else if (lhs->mdy && rhs->mdy) 
+      else if (lhs->dmy && rhs->dmy) 
         {
           if (lhs->year < rhs->year)               return -1;
           else if (lhs->year > rhs->year)               return 1;
@@ -1220,11 +1233,11 @@ g_date_to_struct_tm (GDate      *d,
   g_return_if_fail (g_date_valid (d));
   g_return_if_fail (tm != NULL);
   
-  if (!d->mdy) 
+  if (!d->dmy) 
     {
-      g_date_update_mdy (d);
+      g_date_update_dmy (d);
     }
-  g_return_if_fail (d->mdy);
+  g_return_if_fail (d->dmy);
   
   /* zero all the irrelevant fields to be sure they're valid */