Skip to content

Commit

Permalink
date handling: handle "AM"/"PM" on time
Browse files Browse the repository at this point in the history
And be a bitmore careful about matching: if we don't recognize a word
or a number, we skip the whole thing, rather than trying the next character
in that word/number.

Finally: since ctime() adds the final '\n', don't add another one in test-date.
  • Loading branch information
Linus Torvalds committed May 1, 2005
1 parent 198b0fb commit 68849b5
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 9 deletions.
30 changes: 22 additions & 8 deletions date.c
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,15 @@ static int match_string(const char *date, const char *str)
return i;
}

static int skip_alpha(const char *date)
{
int i = 0;
do {
i++;
} while (isalpha(date[i]));
return i;
}

/*
* Parse month, weekday, or timezone name
*/
Expand Down Expand Up @@ -153,8 +162,14 @@ static int match_alpha(const char *date, struct tm *tm, int *offset)
}
}

if (match_string(date, "PM") == 2) {
if (tm->tm_hour > 0 && tm->tm_hour < 12)
tm->tm_hour += 12;
return 2;
}

/* BAD CRAP */
return 0;
return skip_alpha(date);
}

static int is_date(int year, int month, int day, struct tm *tm)
Expand Down Expand Up @@ -332,14 +347,13 @@ static int match_tz(char *date, int *offp)
* a valid minute. We might want to check that the minutes
* are divisible by 30 or something too.
*/
if (min >= 60 || n < 3)
return 0;
if (min < 60 && n > 2) {
offset = hour*60+min;
if (*date == '-')
offset = -offset;

offset = hour*60+min;
if (*date == '-')
offset = -offset;

*offp = offset;
*offp = offset;
}
return end - date;
}

Expand Down
2 changes: 1 addition & 1 deletion test-date.c
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ int main(int argc, char **argv)
memcpy(result, "bad", 4);
parse_date(argv[i], result, sizeof(result));
t = strtoul(result, NULL, 0);
printf("%s -> %s -> %s\n", argv[i], result, ctime(&t));
printf("%s -> %s -> %s", argv[i], result, ctime(&t));
}
return 0;
}

0 comments on commit 68849b5

Please sign in to comment.