b2938d
From e17dadf36f7b4348e59076240c880d0c78b33fa9 Mon Sep 17 00:00:00 2001
b2938d
From: Karl Williamson <khw@cpan.org>
b2938d
Date: Tue, 22 Sep 2020 08:47:52 -0600
b2938d
Subject: [PATCH] sv.h: sv_collxfrm didn't work properly
b2938d
MIME-Version: 1.0
b2938d
Content-Type: text/plain; charset=UTF-8
b2938d
Content-Transfer-Encoding: 8bit
b2938d
b2938d
It is supposed to be a wrapper for sv_collxfrm_flags, but it was just
b2938d
calling sv_cmp_flags instead.  The consequences are none except under
b2938d
'use locale' in which case you always got the C locale.  I did not add
b2938d
tests, because it is really a pain to write portable locale tests, and
b2938d
this doesn't seem to be much used.  In core the '_flags' form was always
b2938d
used.
b2938d
b2938d
Signed-off-by: Petr Písař <ppisar@redhat.com>
b2938d
---
b2938d
 sv.h | 2 +-
b2938d
 1 file changed, 1 insertion(+), 1 deletion(-)
b2938d
b2938d
diff --git a/sv.h b/sv.h
b2938d
index 19ce718ac3..44414b35a9 100644
b2938d
--- a/sv.h
b2938d
+++ b/sv.h
b2938d
@@ -2045,7 +2045,7 @@ Like C<sv_catsv> but doesn't process magic.
b2938d
 #define sv_eq(sv1, sv2) sv_eq_flags(sv1, sv2, SV_GMAGIC)
b2938d
 #define sv_cmp(sv1, sv2) sv_cmp_flags(sv1, sv2, SV_GMAGIC)
b2938d
 #define sv_cmp_locale(sv1, sv2) sv_cmp_locale_flags(sv1, sv2, SV_GMAGIC)
b2938d
-#define sv_collxfrm(sv, nxp) sv_cmp_flags(sv, nxp, SV_GMAGIC)
b2938d
+#define sv_collxfrm(sv, nxp) sv_collxfrm_flags(sv, nxp, SV_GMAGIC)
b2938d
 #define sv_2bool(sv) sv_2bool_flags(sv, SV_GMAGIC)
b2938d
 #define sv_2bool_nomg(sv) sv_2bool_flags(sv, 0)
b2938d
 #define sv_insert(bigstr, offset, len, little, littlelen)		\
b2938d
-- 
b2938d
2.25.4
b2938d