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.

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);