Merge branch 'jk/date-c-double-semicolon'
authorJunio C Hamano <gitster@pobox.com>
Wed, 30 Oct 2013 19:11:01 +0000 (12:11 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 30 Oct 2013 19:11:01 +0000 (12:11 -0700)
* jk/date-c-double-semicolon:
drop redundant semicolon in empty while

1  2 
date.c
diff --combined date.c
index 29f15404feab0d98f377b312a3fcf5d8888fdb0a,21a538678205eb2bb9b368679383754e83d138ce..83b4166344b31ea615603acaeb95a873f2c05077
--- 1/date.c
--- 2/date.c
+++ b/date.c
@@@ -4,12 -4,12 +4,12 @@@
   * Copyright (C) Linus Torvalds, 2005
   */
  
 -#include <time.h>
 -#include <sys/time.h>
 -
  #include "cache.h"
  
 -static time_t my_mktime(struct tm *tm)
 +/*
 + * This is like mktime, but without normalization of tm_wday and tm_yday.
 + */
 +static time_t tm_to_time_t(const struct tm *tm)
  {
        static const int mdays[] = {
            0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334
@@@ -24,8 -24,6 +24,8 @@@
                return -1;
        if (month < 2 || (year + 2) % 4)
                day--;
 +      if (tm->tm_hour < 0 || tm->tm_min < 0 || tm->tm_sec < 0)
 +              return -1;
        return (year * 365 + (year + 1) / 4 + mdays[month] + day) * 24*60*60UL +
                tm->tm_hour * 60*60 + tm->tm_min * 60 + tm->tm_sec;
  }
@@@ -39,16 -37,6 +39,16 @@@ static const char *weekday_names[] = 
        "Sundays", "Mondays", "Tuesdays", "Wednesdays", "Thursdays", "Fridays", "Saturdays"
  };
  
 +static time_t gm_time_t(unsigned long time, int tz)
 +{
 +      int minutes;
 +
 +      minutes = tz < 0 ? -tz : tz;
 +      minutes = (minutes / 100)*60 + (minutes % 100);
 +      minutes = tz < 0 ? -minutes : minutes;
 +      return time + minutes * 60;
 +}
 +
  /*
   * The "tz" thing is passed in as this strange "decimal parse of tz"
   * thing, which means that tz -0100 is passed in as the integer -100,
   */
  static struct tm *time_to_tm(unsigned long time, int tz)
  {
 -      time_t t;
 -      int minutes;
 -
 -      minutes = tz < 0 ? -tz : tz;
 -      minutes = (minutes / 100)*60 + (minutes % 100);
 -      minutes = tz < 0 ? -minutes : minutes;
 -      t = time + minutes * 60;
 +      time_t t = gm_time_t(time, tz);
        return gmtime(&t);
  }
  
 -const char *show_date(unsigned long time, int tz)
 +/*
 + * What value of "tz" was in effect back then at "time" in the
 + * local timezone?
 + */
 +static int local_tzoffset(unsigned long time)
  {
 -      struct tm *tm;
 -      static char timebuf[200];
 +      time_t t, t_local;
 +      struct tm tm;
 +      int offset, eastwest;
 +
 +      t = time;
 +      localtime_r(&t, &tm);
 +      t_local = tm_to_time_t(&tm);
 +
 +      if (t_local < t) {
 +              eastwest = -1;
 +              offset = t - t_local;
 +      } else {
 +              eastwest = 1;
 +              offset = t_local - t;
 +      }
 +      offset /= 60; /* in minutes */
 +      offset = (offset % 60) + ((offset / 60) * 100);
 +      return offset * eastwest;
 +}
  
 -      tm = time_to_tm(time, tz);
 -      if (!tm)
 -              return NULL;
 -      sprintf(timebuf, "%.3s %.3s %d %02d:%02d:%02d %d %+05d",
 -              weekday_names[tm->tm_wday],
 -              month_names[tm->tm_mon],
 -              tm->tm_mday,
 -              tm->tm_hour, tm->tm_min, tm->tm_sec,
 -              tm->tm_year + 1900, tz);
 -      return timebuf;
 +void show_date_relative(unsigned long time, int tz,
 +                             const struct timeval *now,
 +                             struct strbuf *timebuf)
 +{
 +      unsigned long diff;
 +      if (now->tv_sec < time) {
 +              strbuf_addstr(timebuf, _("in the future"));
 +              return;
 +      }
 +      diff = now->tv_sec - time;
 +      if (diff < 90) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu second ago", "%lu seconds ago", diff), diff);
 +              return;
 +      }
 +      /* Turn it into minutes */
 +      diff = (diff + 30) / 60;
 +      if (diff < 90) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu minute ago", "%lu minutes ago", diff), diff);
 +              return;
 +      }
 +      /* Turn it into hours */
 +      diff = (diff + 30) / 60;
 +      if (diff < 36) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu hour ago", "%lu hours ago", diff), diff);
 +              return;
 +      }
 +      /* We deal with number of days from here on */
 +      diff = (diff + 12) / 24;
 +      if (diff < 14) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu day ago", "%lu days ago", diff), diff);
 +              return;
 +      }
 +      /* Say weeks for the past 10 weeks or so */
 +      if (diff < 70) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu week ago", "%lu weeks ago", (diff + 3) / 7),
 +                       (diff + 3) / 7);
 +              return;
 +      }
 +      /* Say months for the past 12 months or so */
 +      if (diff < 365) {
 +              strbuf_addf(timebuf,
 +                       Q_("%lu month ago", "%lu months ago", (diff + 15) / 30),
 +                       (diff + 15) / 30);
 +              return;
 +      }
 +      /* Give years and months for 5 years or so */
 +      if (diff < 1825) {
 +              unsigned long totalmonths = (diff * 12 * 2 + 365) / (365 * 2);
 +              unsigned long years = totalmonths / 12;
 +              unsigned long months = totalmonths % 12;
 +              if (months) {
 +                      struct strbuf sb = STRBUF_INIT;
 +                      strbuf_addf(&sb, Q_("%lu year", "%lu years", years), years);
 +                      /* TRANSLATORS: "%s" is "<n> years" */
 +                      strbuf_addf(timebuf,
 +                               Q_("%s, %lu month ago", "%s, %lu months ago", months),
 +                               sb.buf, months);
 +                      strbuf_release(&sb);
 +              } else
 +                      strbuf_addf(timebuf,
 +                               Q_("%lu year ago", "%lu years ago", years), years);
 +              return;
 +      }
 +      /* Otherwise, just years. Centuries is probably overkill. */
 +      strbuf_addf(timebuf,
 +               Q_("%lu year ago", "%lu years ago", (diff + 183) / 365),
 +               (diff + 183) / 365);
  }
  
 -const char *show_rfc2822_date(unsigned long time, int tz)
 +const char *show_date(unsigned long time, int tz, enum date_mode mode)
  {
        struct tm *tm;
 -      static char timebuf[200];
 +      static struct strbuf timebuf = STRBUF_INIT;
 +
 +      if (mode == DATE_RAW) {
 +              strbuf_reset(&timebuf);
 +              strbuf_addf(&timebuf, "%lu %+05d", time, tz);
 +              return timebuf.buf;
 +      }
 +
 +      if (mode == DATE_RELATIVE) {
 +              struct timeval now;
 +
 +              strbuf_reset(&timebuf);
 +              gettimeofday(&now, NULL);
 +              show_date_relative(time, tz, &now, &timebuf);
 +              return timebuf.buf;
 +      }
 +
 +      if (mode == DATE_LOCAL)
 +              tz = local_tzoffset(time);
  
        tm = time_to_tm(time, tz);
        if (!tm)
                return NULL;
 -      sprintf(timebuf, "%.3s, %d %.3s %d %02d:%02d:%02d %+05d",
 -              weekday_names[tm->tm_wday], tm->tm_mday,
 -              month_names[tm->tm_mon], tm->tm_year + 1900,
 -              tm->tm_hour, tm->tm_min, tm->tm_sec, tz);
 -      return timebuf;
 +
 +      strbuf_reset(&timebuf);
 +      if (mode == DATE_SHORT)
 +              strbuf_addf(&timebuf, "%04d-%02d-%02d", tm->tm_year + 1900,
 +                              tm->tm_mon + 1, tm->tm_mday);
 +      else if (mode == DATE_ISO8601)
 +              strbuf_addf(&timebuf, "%04d-%02d-%02d %02d:%02d:%02d %+05d",
 +                              tm->tm_year + 1900,
 +                              tm->tm_mon + 1,
 +                              tm->tm_mday,
 +                              tm->tm_hour, tm->tm_min, tm->tm_sec,
 +                              tz);
 +      else if (mode == DATE_RFC2822)
 +              strbuf_addf(&timebuf, "%.3s, %d %.3s %d %02d:%02d:%02d %+05d",
 +                      weekday_names[tm->tm_wday], tm->tm_mday,
 +                      month_names[tm->tm_mon], tm->tm_year + 1900,
 +                      tm->tm_hour, tm->tm_min, tm->tm_sec, tz);
 +      else
 +              strbuf_addf(&timebuf, "%.3s %.3s %d %02d:%02d:%02d %d%c%+05d",
 +                              weekday_names[tm->tm_wday],
 +                              month_names[tm->tm_mon],
 +                              tm->tm_mday,
 +                              tm->tm_hour, tm->tm_min, tm->tm_sec,
 +                              tm->tm_year + 1900,
 +                              (mode == DATE_LOCAL) ? 0 : ' ',
 +                              tz);
 +      return timebuf.buf;
  }
  
  /*
@@@ -247,7 -118,6 +247,7 @@@ static const struct 
  
        { "GMT",    0, 0, },    /* Greenwich Mean */
        { "UTC",    0, 0, },    /* Universal (Coordinated) */
 +      { "Z",      0, 0, },    /* Zulu, alias for UTC */
  
        { "WET",    0, 0, },    /* Western European */
        { "BST",    0, 1, },    /* British Summer */
        { "EAST", +10, 0, },    /* Eastern Australian Standard */
        { "EADT", +10, 1, },    /* Eastern Australian Daylight */
        { "GST",  +10, 0, },    /* Guam Standard, USSR Zone 9 */
 -      { "NZT",  +11, 0, },    /* New Zealand */
 -      { "NZST", +11, 0, },    /* New Zealand Standard */
 -      { "NZDT", +11, 1, },    /* New Zealand Daylight */
 +      { "NZT",  +12, 0, },    /* New Zealand */
 +      { "NZST", +12, 0, },    /* New Zealand Standard */
 +      { "NZDT", +12, 1, },    /* New Zealand Daylight */
        { "IDLE", +12, 0, },    /* International Date Line East */
  };
  
@@@ -324,7 -194,7 +324,7 @@@ static int match_alpha(const char *date
  
        for (i = 0; i < ARRAY_SIZE(timezone_names); i++) {
                int match = match_string(date, timezone_names[i].name);
 -              if (match >= 3) {
 +              if (match >= 3 || match == strlen(timezone_names[i].name)) {
                        int off = timezone_names[i].offset;
  
                        /* This is bogus, but we like summer */
        }
  
        if (match_string(date, "PM") == 2) {
 -              if (tm->tm_hour > 0 && tm->tm_hour < 12)
 -                      tm->tm_hour += 12;
 +              tm->tm_hour = (tm->tm_hour % 12) + 12;
 +              return 2;
 +      }
 +
 +      if (match_string(date, "AM") == 2) {
 +              tm->tm_hour = (tm->tm_hour % 12) + 0;
                return 2;
        }
  
@@@ -377,13 -243,13 +377,13 @@@ static int is_date(int year, int month
                if (!now_tm)
                        return 1;
  
 -              specified = my_mktime(r);
 +              specified = tm_to_time_t(r);
  
                /* Be it commit time or author time, it does not make
                 * sense to specify timestamp way into the future.  Make
                 * sure it is not later than ten days from now...
                 */
 -              if (now + 10*24*3600 < specified)
 +              if ((specified != -1) && (now + 10*24*3600 < specified))
                        return 0;
                tm->tm_mon = r->tm_mon;
                tm->tm_mday = r->tm_mday;
@@@ -455,22 -321,7 +455,22 @@@ static int match_multi_number(unsigned 
  }
  
  /*
 - * We've seen a digit. Time? Year? Date? 
 + * Have we filled in any part of the time/date yet?
 + * We just do a binary 'and' to see if the sign bit
 + * is set in all the values.
 + */
 +static inline int nodate(struct tm *tm)
 +{
 +      return (tm->tm_year &
 +              tm->tm_mon &
 +              tm->tm_mday &
 +              tm->tm_hour &
 +              tm->tm_min &
 +              tm->tm_sec) < 0;
 +}
 +
 +/*
 + * We've seen a digit. Time? Year? Date?
   */
  static int match_digit(const char *date, struct tm *tm, int *offset, int *tm_gmt)
  {
        num = strtoul(date, &end, 10);
  
        /*
 -       * Seconds since 1970? We trigger on that for anything after Jan 1, 2000
 +       * Seconds since 1970? We trigger on that for any numbers with
 +       * more than 8 digits. This is because we don't want to rule out
 +       * numbers like 20070606 as a YYYYMMDD date.
         */
 -      if (num > 946684800) {
 +      if (num >= 100000000 && nodate(tm)) {
                time_t time = num;
                if (gmtime_r(&time, tm)) {
                        *tm_gmt = 1;
                return n;
        }
  
 +      /*
 +       * Ignore lots of numerals. We took care of 4-digit years above.
 +       * Days or months must be one or two digits.
 +       */
 +      if (n > 2)
 +              return n;
 +
        /*
         * NOTE! We will give precedence to day-of-month over month or
         * year numbers in the 1-12 range. So 05 is always "mday 5",
                }
        }
  
 -      if (num > 0 && num < 32) {
 -              tm->tm_mday = num;
 -      } else if (num > 1900) {
 -              tm->tm_year = num - 1900;
 -      } else if (num > 70) {
 -              tm->tm_year = num;
 -      } else if (num > 0 && num < 13) {
 +      if (num > 0 && num < 13 && tm->tm_mon < 0)
                tm->tm_mon = num-1;
 -      }
 -              
 +
        return n;
  }
  
  static int match_tz(const char *date, int *offp)
  {
        char *end;
 -      int offset = strtoul(date+1, &end, 10);
 -      int min, hour;
 -      int n = end - date - 1;
 +      int hour = strtoul(date + 1, &end, 10);
 +      int n = end - (date + 1);
 +      int min = 0;
  
 -      min = offset % 100;
 -      hour = offset / 100;
 +      if (n == 4) {
 +              /* hhmm */
 +              min = hour % 100;
 +              hour = hour / 100;
 +      } else if (n != 2) {
 +              min = 99; /* random crap */
 +      } else if (*end == ':') {
 +              /* hh:mm? */
 +              min = strtoul(end + 1, &end, 10);
 +              if (end - (date + 1) != 5)
 +                      min = 99; /* random crap */
 +      } /* otherwise we parsed "hh" */
  
        /*
 -       * Don't accept any random crap.. At least 3 digits, and
 -       * a valid minute. We might want to check that the minutes
 -       * are divisible by 30 or something too.
 +       * Don't accept any random crap. Even though some places have
 +       * offset larger than 12 hours (e.g. Pacific/Kiritimati is at
 +       * UTC+14), there is something wrong if hour part is much
 +       * larger than that. We might also want to check that the
 +       * minutes are divisible by 15 or something too. (Offset of
 +       * Kathmandu, Nepal is UTC+5:45)
         */
 -      if (min < 60 && n > 2) {
 -              offset = hour*60+min;
 +      if (min < 60 && hour < 24) {
 +              int offset = hour * 60 + min;
                if (*date == '-')
                        offset = -offset;
 -
                *offp = offset;
        }
        return end - date;
@@@ -614,61 -451,22 +614,61 @@@ static int date_string(unsigned long da
        return snprintf(buf, len, "%lu %c%02d%02d", date, sign, offset / 60, offset % 60);
  }
  
 +/*
 + * Parse a string like "0 +0000" as ancient timestamp near epoch, but
 + * only when it appears not as part of any other string.
 + */
 +static int match_object_header_date(const char *date, unsigned long *timestamp, int *offset)
 +{
 +      char *end;
 +      unsigned long stamp;
 +      int ofs;
 +
 +      if (*date < '0' || '9' < *date)
 +              return -1;
 +      stamp = strtoul(date, &end, 10);
 +      if (*end != ' ' || stamp == ULONG_MAX || (end[1] != '+' && end[1] != '-'))
 +              return -1;
 +      date = end + 2;
 +      ofs = strtol(date, &end, 10);
 +      if ((*end != '\0' && (*end != '\n')) || end != date + 4)
 +              return -1;
 +      ofs = (ofs / 100) * 60 + (ofs % 100);
 +      if (date[-1] == '-')
 +              ofs = -ofs;
 +      *timestamp = stamp;
 +      *offset = ofs;
 +      return 0;
 +}
 +
  /* Gr. strptime is crap for this; it doesn't have a way to require RFC2822
     (i.e. English) day/month names, and it doesn't work correctly with %z. */
 -int parse_date(const char *date, char *result, int maxlen)
 +int parse_date_basic(const char *date, unsigned long *timestamp, int *offset)
  {
        struct tm tm;
 -      int offset, tm_gmt;
 -      time_t then;
 +      int tm_gmt;
 +      unsigned long dummy_timestamp;
 +      int dummy_offset;
 +
 +      if (!timestamp)
 +              timestamp = &dummy_timestamp;
 +      if (!offset)
 +              offset = &dummy_offset;
  
        memset(&tm, 0, sizeof(tm));
        tm.tm_year = -1;
        tm.tm_mon = -1;
        tm.tm_mday = -1;
        tm.tm_isdst = -1;
 -      offset = -1;
 +      tm.tm_hour = -1;
 +      tm.tm_min = -1;
 +      tm.tm_sec = -1;
 +      *offset = -1;
        tm_gmt = 0;
  
 +      if (*date == '@' &&
 +          !match_object_header_date(date + 1, timestamp, offset))
 +              return 0; /* success */
        for (;;) {
                int match = 0;
                unsigned char c = *date;
                        break;
  
                if (isalpha(c))
 -                      match = match_alpha(date, &tm, &offset);
 +                      match = match_alpha(date, &tm, offset);
                else if (isdigit(c))
 -                      match = match_digit(date, &tm, &offset, &tm_gmt);
 +                      match = match_digit(date, &tm, offset, &tm_gmt);
                else if ((c == '-' || c == '+') && isdigit(date[1]))
 -                      match = match_tz(date, &offset);
 +                      match = match_tz(date, offset);
  
                if (!match) {
                        /* BAD CRAP */
                        match = 1;
 -              }       
 +              }
  
                date += match;
        }
  
        /* mktime uses local timezone */
 -      then = my_mktime(&tm); 
 -      if (offset == -1)
 -              offset = (then - mktime(&tm)) / 60;
 +      *timestamp = tm_to_time_t(&tm);
 +      if (*offset == -1) {
 +              time_t temp_time = mktime(&tm);
 +              if ((time_t)*timestamp > temp_time) {
 +                      *offset = ((time_t)*timestamp - temp_time) / 60;
 +              } else {
 +                      *offset = -(int)((temp_time - (time_t)*timestamp) / 60);
 +              }
 +      }
  
 -      if (then == -1)
 +      if (*timestamp == -1)
                return -1;
  
        if (!tm_gmt)
 -              then -= offset * 60;
 -      return date_string(then, offset, result, maxlen);
 +              *timestamp -= *offset * 60;
 +      return 0; /* success */
 +}
 +
 +int parse_expiry_date(const char *date, unsigned long *timestamp)
 +{
 +      int errors = 0;
 +
 +      if (!strcmp(date, "never") || !strcmp(date, "false"))
 +              *timestamp = 0;
 +      else if (!strcmp(date, "all") || !strcmp(date, "now"))
 +              /*
 +               * We take over "now" here, which usually translates
 +               * to the current timestamp.  This is because the user
 +               * really means to expire everything she has done in
 +               * the past, and by definition reflogs are the record
 +               * of the past, and there is nothing from the future
 +               * to be kept.
 +               */
 +              *timestamp = ULONG_MAX;
 +      else
 +              *timestamp = approxidate_careful(date, &errors);
 +
 +      return errors;
 +}
 +
 +int parse_date(const char *date, char *result, int maxlen)
 +{
 +      unsigned long timestamp;
 +      int offset;
 +      if (parse_date_basic(date, &timestamp, &offset))
 +              return -1;
 +      return date_string(timestamp, offset, result, maxlen);
 +}
 +
 +enum date_mode parse_date_format(const char *format)
 +{
 +      if (!strcmp(format, "relative"))
 +              return DATE_RELATIVE;
 +      else if (!strcmp(format, "iso8601") ||
 +               !strcmp(format, "iso"))
 +              return DATE_ISO8601;
 +      else if (!strcmp(format, "rfc2822") ||
 +               !strcmp(format, "rfc"))
 +              return DATE_RFC2822;
 +      else if (!strcmp(format, "short"))
 +              return DATE_SHORT;
 +      else if (!strcmp(format, "local"))
 +              return DATE_LOCAL;
 +      else if (!strcmp(format, "default"))
 +              return DATE_NORMAL;
 +      else if (!strcmp(format, "raw"))
 +              return DATE_RAW;
 +      else
 +              die("unknown date format %s", format);
  }
  
  void datestamp(char *buf, int bufsize)
  
        time(&now);
  
 -      offset = my_mktime(localtime(&now)) - now;
 +      offset = tm_to_time_t(localtime(&now)) - now;
        offset /= 60;
  
        date_string(now, offset, buf, bufsize);
  }
  
 -static void update_tm(struct tm *tm, unsigned long sec)
 +/*
 + * Relative time update (eg "2 days ago").  If we haven't set the time
 + * yet, we need to set it from current time.
 + */
 +static unsigned long update_tm(struct tm *tm, struct tm *now, unsigned long sec)
  {
 -      time_t n = mktime(tm) - sec;
 +      time_t n;
 +
 +      if (tm->tm_mday < 0)
 +              tm->tm_mday = now->tm_mday;
 +      if (tm->tm_mon < 0)
 +              tm->tm_mon = now->tm_mon;
 +      if (tm->tm_year < 0) {
 +              tm->tm_year = now->tm_year;
 +              if (tm->tm_mon > now->tm_mon)
 +                      tm->tm_year--;
 +      }
 +
 +      n = mktime(tm) - sec;
        localtime_r(&n, tm);
 +      return n;
  }
  
 -static void date_yesterday(struct tm *tm, int *num)
 +static void date_now(struct tm *tm, struct tm *now, int *num)
  {
 -      update_tm(tm, 24*60*60);
 +      update_tm(tm, now, 0);
  }
  
 -static void date_time(struct tm *tm, int hour)
 +static void date_yesterday(struct tm *tm, struct tm *now, int *num)
 +{
 +      update_tm(tm, now, 24*60*60);
 +}
 +
 +static void date_time(struct tm *tm, struct tm *now, int hour)
  {
        if (tm->tm_hour < hour)
 -              date_yesterday(tm, NULL);
 +              date_yesterday(tm, now, NULL);
        tm->tm_hour = hour;
        tm->tm_min = 0;
        tm->tm_sec = 0;
  }
  
 -static void date_midnight(struct tm *tm, int *num)
 +static void date_midnight(struct tm *tm, struct tm *now, int *num)
 +{
 +      date_time(tm, now, 0);
 +}
 +
 +static void date_noon(struct tm *tm, struct tm *now, int *num)
 +{
 +      date_time(tm, now, 12);
 +}
 +
 +static void date_tea(struct tm *tm, struct tm *now, int *num)
  {
 -      date_time(tm, 0);
 +      date_time(tm, now, 17);
  }
  
 -static void date_noon(struct tm *tm, int *num)
 +static void date_pm(struct tm *tm, struct tm *now, int *num)
  {
 -      date_time(tm, 12);
 +      int hour, n = *num;
 +      *num = 0;
 +
 +      hour = tm->tm_hour;
 +      if (n) {
 +              hour = n;
 +              tm->tm_min = 0;
 +              tm->tm_sec = 0;
 +      }
 +      tm->tm_hour = (hour % 12) + 12;
 +}
 +
 +static void date_am(struct tm *tm, struct tm *now, int *num)
 +{
 +      int hour, n = *num;
 +      *num = 0;
 +
 +      hour = tm->tm_hour;
 +      if (n) {
 +              hour = n;
 +              tm->tm_min = 0;
 +              tm->tm_sec = 0;
 +      }
 +      tm->tm_hour = (hour % 12);
  }
  
 -static void date_tea(struct tm *tm, int *num)
 +static void date_never(struct tm *tm, struct tm *now, int *num)
  {
 -      date_time(tm, 17);
 +      time_t n = 0;
 +      localtime_r(&n, tm);
  }
  
  static const struct special {
        const char *name;
 -      void (*fn)(struct tm *, int *);
 +      void (*fn)(struct tm *, struct tm *, int *);
  } special[] = {
        { "yesterday", date_yesterday },
        { "noon", date_noon },
        { "midnight", date_midnight },
        { "tea", date_tea },
 +      { "PM", date_pm },
 +      { "AM", date_am },
 +      { "never", date_never },
 +      { "now", date_now },
        { NULL }
  };
  
@@@ -898,23 -577,22 +898,23 @@@ static const struct typelen 
        { "days", 24*60*60 },
        { "weeks", 7*24*60*60 },
        { NULL }
 -};    
 +};
  
 -static const char *approxidate_alpha(const char *date, struct tm *tm, int *num)
 +static const char *approxidate_alpha(const char *date, struct tm *tm, struct tm *now, int *num, int *touched)
  {
        const struct typelen *tl;
        const struct special *s;
        const char *end = date;
        int i;
  
-       while (isalpha(*++end));
+       while (isalpha(*++end))
                ;
  
        for (i = 0; i < 12; i++) {
                int match = match_string(date, month_names[i]);
                if (match >= 3) {
                        tm->tm_mon = i;
 +                      *touched = 1;
                        return end;
                }
        }
        for (s = special; s->name; s++) {
                int len = strlen(s->name);
                if (match_string(date, s->name) == len) {
 -                      s->fn(tm, num);
 +                      s->fn(tm, now, num);
 +                      *touched = 1;
                        return end;
                }
        }
                        int len = strlen(number_name[i]);
                        if (match_string(date, number_name[i]) == len) {
                                *num = i;
 +                              *touched = 1;
                                return end;
                        }
                }
 -              if (match_string(date, "last") == 4)
 +              if (match_string(date, "last") == 4) {
                        *num = 1;
 +                      *touched = 1;
 +              }
                return end;
        }
  
        while (tl->type) {
                int len = strlen(tl->type);
                if (match_string(date, tl->type) >= len-1) {
 -                      update_tm(tm, tl->length * *num);
 +                      update_tm(tm, now, tl->length * *num);
                        *num = 0;
 +                      *touched = 1;
                        return end;
                }
                tl++;
                                n++;
                        diff += 7*n;
  
 -                      update_tm(tm, diff * 24 * 60 * 60);
 +                      update_tm(tm, now, diff * 24 * 60 * 60);
 +                      *touched = 1;
                        return end;
                }
        }
  
        if (match_string(date, "months") >= 5) {
 -              int n = tm->tm_mon - *num;
 +              int n;
 +              update_tm(tm, now, 0); /* fill in date fields if needed */
 +              n = tm->tm_mon - *num;
                *num = 0;
                while (n < 0) {
                        n += 12;
                        tm->tm_year--;
                }
                tm->tm_mon = n;
 +              *touched = 1;
                return end;
        }
  
        if (match_string(date, "years") >= 4) {
 +              update_tm(tm, now, 0); /* fill in date fields if needed */
                tm->tm_year -= *num;
                *num = 0;
 +              *touched = 1;
                return end;
        }
  
        return end;
  }
  
 -unsigned long approxidate(const char *date)
 +static const char *approxidate_digit(const char *date, struct tm *tm, int *num)
 +{
 +      char *end;
 +      unsigned long number = strtoul(date, &end, 10);
 +
 +      switch (*end) {
 +      case ':':
 +      case '.':
 +      case '/':
 +      case '-':
 +              if (isdigit(end[1])) {
 +                      int match = match_multi_number(number, *end, date, end, tm);
 +                      if (match)
 +                              return date + match;
 +              }
 +      }
 +
 +      /* Accept zero-padding only for small numbers ("Dec 02", never "Dec 0002") */
 +      if (date[0] != '0' || end - date <= 2)
 +              *num = number;
 +      return end;
 +}
 +
 +/*
 + * Do we have a pending number at the end, or when
 + * we see a new one? Let's assume it's a month day,
 + * as in "Dec 6, 1992"
 + */
 +static void pending_number(struct tm *tm, int *num)
 +{
 +      int number = *num;
 +
 +      if (number) {
 +              *num = 0;
 +              if (tm->tm_mday < 0 && number < 32)
 +                      tm->tm_mday = number;
 +              else if (tm->tm_mon < 0 && number < 13)
 +                      tm->tm_mon = number-1;
 +              else if (tm->tm_year < 0) {
 +                      if (number > 1969 && number < 2100)
 +                              tm->tm_year = number - 1900;
 +                      else if (number > 69 && number < 100)
 +                              tm->tm_year = number;
 +                      else if (number < 38)
 +                              tm->tm_year = 100 + number;
 +                      /* We screw up for number = 00 ? */
 +              }
 +      }
 +}
 +
 +static unsigned long approxidate_str(const char *date,
 +                                   const struct timeval *tv,
 +                                   int *error_ret)
  {
        int number = 0;
 +      int touched = 0;
        struct tm tm, now;
 -      struct timeval tv;
 -      char buffer[50];
 -
 -      if (parse_date(date, buffer, sizeof(buffer)) > 0)
 -              return strtoul(buffer, NULL, 10);
 +      time_t time_sec;
  
 -      gettimeofday(&tv, NULL);
 -      localtime_r(&tv.tv_sec, &tm);
 +      time_sec = tv->tv_sec;
 +      localtime_r(&time_sec, &tm);
        now = tm;
 +
 +      tm.tm_year = -1;
 +      tm.tm_mon = -1;
 +      tm.tm_mday = -1;
 +
        for (;;) {
                unsigned char c = *date;
                if (!c)
                        break;
                date++;
                if (isdigit(c)) {
 -                      char *end;
 -                      number = strtoul(date-1, &end, 10);
 -                      date = end;
 +                      pending_number(&tm, &number);
 +                      date = approxidate_digit(date-1, &tm, &number);
 +                      touched = 1;
                        continue;
                }
                if (isalpha(c))
 -                      date = approxidate_alpha(date-1, &tm, &number);
 +                      date = approxidate_alpha(date-1, &tm, &now, &number, &touched);
 +      }
 +      pending_number(&tm, &number);
 +      if (!touched)
 +              *error_ret = 1;
 +      return update_tm(&tm, &now, 0);
 +}
 +
 +unsigned long approxidate_relative(const char *date, const struct timeval *tv)
 +{
 +      unsigned long timestamp;
 +      int offset;
 +      int errors = 0;
 +
 +      if (!parse_date_basic(date, &timestamp, &offset))
 +              return timestamp;
 +      return approxidate_str(date, tv, &errors);
 +}
 +
 +unsigned long approxidate_careful(const char *date, int *error_ret)
 +{
 +      struct timeval tv;
 +      unsigned long timestamp;
 +      int offset;
 +      int dummy = 0;
 +      if (!error_ret)
 +              error_ret = &dummy;
 +
 +      if (!parse_date_basic(date, &timestamp, &offset)) {
 +              *error_ret = 0;
 +              return timestamp;
        }
 -      if (number > 0 && number < 32)
 -              tm.tm_mday = number;
 -      if (tm.tm_mon > now.tm_mon && tm.tm_year == now.tm_year)
 -              tm.tm_year--;
 -      return mktime(&tm);
 +
 +      gettimeofday(&tv, NULL);
 +      return approxidate_str(date, &tv, error_ret);
  }