Fixes bug of having default argument for TimeRE's __init__ that caused the

LocaleTime instance to only be created once and thus not be recreated when
the locale changed.
This commit is contained in:
Brett Cannon 2003-07-24 20:02:28 +00:00
parent 9a71475e9e
commit 2c24d42d32
1 changed files with 5 additions and 2 deletions

View File

@ -292,7 +292,7 @@ class LocaleTime(object):
class TimeRE(dict): class TimeRE(dict):
"""Handle conversion from format directives to regexes.""" """Handle conversion from format directives to regexes."""
def __init__(self, locale_time=LocaleTime()): def __init__(self, locale_time=None):
"""Init inst with non-locale regexes and store LocaleTime object.""" """Init inst with non-locale regexes and store LocaleTime object."""
#XXX: Does 'Y' need to worry about having less or more than 4 digits? #XXX: Does 'Y' need to worry about having less or more than 4 digits?
base = super(TimeRE, self) base = super(TimeRE, self)
@ -311,7 +311,10 @@ class TimeRE(dict):
'y': r"(?P<y>\d\d)", 'y': r"(?P<y>\d\d)",
'Y': r"(?P<Y>\d\d\d\d)"}) 'Y': r"(?P<Y>\d\d\d\d)"})
base.__setitem__('W', base.__getitem__('U')) base.__setitem__('W', base.__getitem__('U'))
self.locale_time = locale_time if locale_time:
self.locale_time = locale_time
else:
self.locale_time = LocaleTime()
def __getitem__(self, fetch): def __getitem__(self, fetch):
"""Try to fetch regex; if it does not exist, construct it.""" """Try to fetch regex; if it does not exist, construct it."""