From 6e75277b122c47ca63d9520780291cf4bc7251cf Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Tue, 28 Feb 2023 12:10:40 +0100 Subject: [PATCH] Cygwin: __wscollate_range_cmp: fix incorrect comment The comment that the first arg must be the pattern was added during development, before it turned out that __wscollate_range_cmp can be implemented in an order independent way. Better explain why this function uses pointers to strings. Signed-off-by: Corinna Vinschen --- winsup/cygwin/nlsfuncs.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/winsup/cygwin/nlsfuncs.cc b/winsup/cygwin/nlsfuncs.cc index dd0fc8dc5..04d7072e7 100644 --- a/winsup/cygwin/nlsfuncs.cc +++ b/winsup/cygwin/nlsfuncs.cc @@ -1212,8 +1212,8 @@ __wcollate_range_cmp (wint_t c1, wint_t c2) /* Not so much BSD. Used from glob.cc, fnmatch.c and regcomp.c. - First arg is always from pattern space, second arg is the tested string. - len is the length of the pattern in the first arg. */ + The args are pointers to wint_t strings. This allows to compare + against collating symbols. */ extern "C" int __wscollate_range_cmp (wint_t *c1, wint_t *c2, size_t c1len, size_t c2len)