diff --git a/Lib/_strptime.py b/Lib/_strptime.py index b7f75779d7d..97cb551819f 100644 --- a/Lib/_strptime.py +++ b/Lib/_strptime.py @@ -306,22 +306,23 @@ class TimeRE(dict): def __init__(self, locale_time=LocaleTime()): """Init inst with non-locale regexes and store LocaleTime object.""" - # XXX: should 0 be valid for: - # day (d), julian day (j), month (m), and hour12 (I)? - super(TimeRE,self).__init__({ + #XXX: Does 'Y' need to worry about having less or more than 4 digits? + base = super(TimeRE, self) + base.__init__({ # The " \d" option is to make %c from ANSI C work - 'd': r"(?P3[0-1]|[0-2]\d|\d| \d)", + 'd': r"(?P3[0-1]|[1-2]\d|0[1-9]|[1-9]| [1-9])", 'H': r"(?P2[0-3]|[0-1]\d|\d)", - 'I': r"(?P0\d|1[0-2]|\d)", - 'j': r"(?P(?:3[0-5]\d|36[0-6])|[0-2]\d\d|\d\d|\d)", - 'm': r"(?P0\d|1[0-2]|\d)", + 'I': r"(?P1[0-2]|0[1-9]|[1-9])", + 'j': r"(?P36[0-6]|3[0-5]\d|[1-2]\d\d|0[1-9]\d|00[1-9]|[1-9]\d|0[1-9]|[1-9])", + 'm': r"(?P1[0-2]|0[1-9]|[1-9])", 'M': r"(?P[0-5]\d|\d)", 'S': r"(?P6[0-1]|[0-5]\d|\d)", 'U': r"(?P5[0-3]|[0-4]\d|\d)", 'w': r"(?P[0-6])", - 'W': r"(?P5[0-3]|[0-4]\d|\d)", # Same as U + # W is set below by using 'U' 'y': r"(?P\d\d)", 'Y': r"(?P\d\d\d\d)"}) + base.__setitem__('W', base.__getitem__('U')) self.locale_time = locale_time def __getitem__(self, fetch):