diff options
| author | Rich Felker <dalias@aerifal.cx> | 2013-07-28 03:41:01 -0400 | 
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2013-07-28 03:41:01 -0400 | 
| commit | 1ae4bc42808111d6b3f50c063de102f162051986 (patch) | |
| tree | bed1400ccc266c4f86c9960e45210da58530b5c8 /src/time/strftime.c | |
| parent | f44e239f9f36c4be049f98d6a3d33644168e40a7 (diff) | |
| download | musl-1ae4bc42808111d6b3f50c063de102f162051986.tar.gz | |
fix semantically incorrect use of LC_GLOBAL_LOCALE
LC_GLOBAL_LOCALE refers to the global locale, controlled by setlocale,
not the thread-local locale in effect which these functions should be
using. neither LC_GLOBAL_LOCALE nor 0 has an argument to the *_l
functions has behavior defined by the standard, but 0 is a more
logical choice for requesting the callee to lookup the current locale.
in the future I may move the current locale lookup the the caller (the
non-_l-suffixed wrapper).
at this point, all of the locale logic is dummied out, so no harm was
done, but it should at least avoid misleading usage.
Diffstat (limited to 'src/time/strftime.c')
| -rw-r--r-- | src/time/strftime.c | 2 | 
1 files changed, 1 insertions, 1 deletions
diff --git a/src/time/strftime.c b/src/time/strftime.c index 96cb996d..3f0ec3e3 100644 --- a/src/time/strftime.c +++ b/src/time/strftime.c @@ -214,7 +214,7 @@ recu_strftime:  size_t strftime(char *restrict s, size_t n, const char *restrict f, const struct tm *restrict tm)  { -	return __strftime_l(s, n, f, tm, LC_GLOBAL_LOCALE); +	return __strftime_l(s, n, f, tm, 0);  }  weak_alias(__strftime_l, strftime_l);  | 
