Re: [9fans] tiny bug in Acme News

[Jonathan Sergent <sergent@IO.COM>]

On Saturday, July 28, 2001, at 08:07 PM, crdevilb@mtu.edu wrote:

> /acme/news/src/news.c:119
> 		if(1 <= j && j <= 31)
> should be
> 		if(1 <= j && j <= 31 && !d)
>
> or it will report the date as the current hour.

There are actually more problems than that if you look at the different
formats in use in an average (?) newsgroup.  There are things with
timezone specifiers and such.  A few months ago, I hacked my fixdate()
to:

char*
fixdate(char *s)
{
         char *f[10], *m, *t, *wd, tmp[40];
         int d, i, j, nf, hh, mm, mn, yr;

         nf = tokenize(s, f, nelem(f));

         wd = nil;
         d = 0;
         m = nil;
         t = nil;
         yr = 0;
         mn = 0;
         for(i=0; i<nf; i++){
                 for(j=0; j<7; j++)
                         if(cistrncmp(day[j], f[i], 3)==0)
                                 wd = day[j];
                 for(j=0; j<12; j++)
                         if(cistrncmp(mon[j], f[i], 3)==0) {
                                 m = mon[j];
                                 mn = j;
                         }

                 while (f[i][0] && f[i][0] == '0')
                         f[i]++;

                 j = atoi(f[i]);
                 if(strchr(f[i], ':'))
                         t = f[i];
                 else if(f[i][0] == '+')
                         continue;
                 else if((d == 0) && (1 <= j && j <= 31))
                         d = j;
                 else if ((d != 0) && (j > 0)) {
                         if (j < 70)
                                 yr = j + 2000;
                         else if (j < 100)
                                 yr = j + 1900;
                         else if (j > 1970)
                                 yr = j;
                 }
         }

         if(d==0 || m==nil || t==nil)
                 return nil;

         hh = strtol(t, 0, 10);
         mm = strtol(strchr(t, ':')+1, 0, 10);

         if (wd == nil) {
                 Tm tm = { 0, mm, hh, d, mn, yr - 1900, 0, 0, "", 0 };
                 if (yr == 0 || mn == 0)
                         return nil;
                 wd = day[localtime(tm2sec(&tm))->wday];
         }

         sprint(tmp, "%s %d %s %d:%.2d", wd, d, m, hh, mm);
         return estrdup(tmp);
}

I might have changed more than this function, but I can't remember.  The
whole thing is a little ugly.