From 2f334ad5c3c1510f5dc6b0a1ecebba2168effba7 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 9 Nov 2006 20:20:23 +0000 Subject: [PATCH] * string/Makefile (tests): Add tst-strxfrm2. * string/tst-strxfrm2.c: New file. * string/strxfrm_l.c (STRXFRM): Do the trailing \1 removal optimization even if needed > n. --- ChangeLog | 8 ++++++++ string/Makefile | 2 +- string/strxfrm_l.c | 11 +++++++---- string/tst-strxfrm2.c | 43 +++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 59 insertions(+), 5 deletions(-) create mode 100644 string/tst-strxfrm2.c diff --git a/ChangeLog b/ChangeLog index bd3bf357d2..7932258837 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2006-11-09 Ulrich Drepper + + * string/Makefile (tests): Add tst-strxfrm2. + * string/tst-strxfrm2.c: New file. + 2006-10-09 Jakub Jelinek * elf/dl-debug.c (_dl_debug_initialize): Check r->r_map for 0 @@ -5,6 +10,9 @@ 2006-11-08 Jakub Jelinek + * string/strxfrm_l.c (STRXFRM): Do the trailing \1 removal + optimization even if needed > n. + * elf/dl-load.c (decompose_rpath): Return bool rather than void. If l->l_name is on inhibit_rpath list, set sps->dirs to -1 and return false, otherwise return true. diff --git a/string/Makefile b/string/Makefile index a84ebebcaa..3e79f4d909 100644 --- a/string/Makefile +++ b/string/Makefile @@ -54,7 +54,7 @@ tests := tester inl-tester noinl-tester testcopy test-ffs \ bug-strncat1 bug-strspn1 bug-strpbrk1 tst-bswap \ tst-strtok tst-strxfrm bug-strcoll1 tst-strfry \ bug-strtok1 $(addprefix test-,$(strop-tests)) \ - bug-envz1 + bug-envz1 tst-strxfrm2 distribute := memcopy.h pagecopy.h tst-svc.expect test-string.h diff --git a/string/strxfrm_l.c b/string/strxfrm_l.c index 5335601919..f158833f05 100644 --- a/string/strxfrm_l.c +++ b/string/strxfrm_l.c @@ -1,4 +1,5 @@ -/* Copyright (C) 1995,96,97,2002, 2004, 2005 Free Software Foundation, Inc. +/* Copyright (C) 1995, 1996, 1997, 2002, 2004, 2005, 2006 + Free Software Foundation, Inc. This file is part of the GNU C Library. Written by Ulrich Drepper , 1995. @@ -96,6 +97,7 @@ STRXFRM (STRING_TYPE *dest, const STRING_TYPE *src, size_t n, __locale_t l) const int32_t *indirect; uint_fast32_t pass; size_t needed; + size_t last_needed; const USTRING_TYPE *usrc; size_t srclen = STRLEN (src); int32_t *idxarr; @@ -197,6 +199,7 @@ STRXFRM (STRING_TYPE *dest, const STRING_TYPE *src, size_t n, __locale_t l) this is true for all of them. */ int position = rule & sort_position; + last_needed = needed; if (position == 0) { for (idxcnt = 0; idxcnt < idxmax; ++idxcnt) @@ -426,11 +429,11 @@ STRXFRM (STRING_TYPE *dest, const STRING_TYPE *src, size_t n, __locale_t l) a `position' rule at the end and if no non-ignored character is found the last \1 byte is immediately followed by a \0 byte signalling this. We can avoid the \1 byte(s). */ - if (needed <= n && needed > 2 && dest[needed - 2] == L('\1')) + if (needed > 2 && needed == last_needed + 1) { /* Remove the \1 byte. */ - --needed; - dest[needed - 1] = L('\0'); + if (--needed < n) + dest[needed - 1] = L('\0'); } /* Free the memory if needed. */ diff --git a/string/tst-strxfrm2.c b/string/tst-strxfrm2.c new file mode 100644 index 0000000000..31fc42cc93 --- /dev/null +++ b/string/tst-strxfrm2.c @@ -0,0 +1,43 @@ +#include +#include +#include + +static int +do_test (void) +{ + int res = 0; + + char buf[10]; + size_t l1 = strxfrm (NULL, "ab", 0); + size_t l2 = strxfrm (buf, "ab", 1); + size_t l3 = strxfrm (buf, "ab", sizeof (buf)); + + if (l1 != l2 || l1 != l3) + { + puts ("C locale test failed"); + res = 1; + } + + if (setlocale (LC_ALL, "de_DE.UTF-8") == NULL) + { + puts ("setlocale failed"); + res = 1; + } + else + { + l1 = strxfrm (NULL, "ab", 0); + l2 = strxfrm (buf, "ab", 1); + l3 = strxfrm (buf, "ab", sizeof (buf)); + + if (l1 != l2 || l1 != l3) + { + puts ("UTF-8 locale test failed"); + res = 1; + } + } + + return res; +} + +#define TEST_FUNCTION do_test () +#include "../test-skeleton.c"