about summary refs log tree commit diff
path: root/src/locale/strcoll.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2013-07-28 03:41:01 -0400
committerRich Felker <dalias@aerifal.cx>2013-07-28 03:41:01 -0400
commit1ae4bc42808111d6b3f50c063de102f162051986 (patch)
treebed1400ccc266c4f86c9960e45210da58530b5c8 /src/locale/strcoll.c
parentf44e239f9f36c4be049f98d6a3d33644168e40a7 (diff)
downloadmusl-1ae4bc42808111d6b3f50c063de102f162051986.tar.gz
musl-1ae4bc42808111d6b3f50c063de102f162051986.tar.xz
musl-1ae4bc42808111d6b3f50c063de102f162051986.zip
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/locale/strcoll.c')
-rw-r--r--src/locale/strcoll.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/locale/strcoll.c b/src/locale/strcoll.c
index 8b84b1e6..39ea1123 100644
--- a/src/locale/strcoll.c
+++ b/src/locale/strcoll.c
@@ -9,7 +9,7 @@ int __strcoll_l(const char *l, const char *r, locale_t loc)
 
 int strcoll(const char *l, const char *r)
 {
-	return __strcoll_l(l, r, LC_GLOBAL_LOCALE);
+	return __strcoll_l(l, r, 0);
 }
 
 weak_alias(__strcoll_l, strcoll_l);