This is the mail archive of the
libc-locales@sourceware.org
mailing list for the GNU libc locales project.
[Bug locale/18589] sort-test.sh fails at random
- From: "cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>
- To: libc-locales at sourceware dot org
- Date: Sat, 10 Oct 2015 00:43:19 +0000
- Subject: [Bug locale/18589] sort-test.sh fails at random
- Auto-submitted: auto-generated
- References: <bug-18589-716 at http dot sourceware dot org/bugzilla/>
https://sourceware.org/bugzilla/show_bug.cgi?id=18589
--- Comment #12 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, release/2.22/master has been updated
via 2b529a82afc15a76185fca16ba7bcc8fbd82d15d (commit)
via c0769d60c1d107d9fb622e80c9d1110deb4fb3b4 (commit)
via ed7f3a2e27e816488d5a8ef570052a5a2c3c515b (commit)
via 6c84109cfa26f35c3dfed3acb97d347361bd5849 (commit)
from 666bb4dc55f5a93538d020024c34abad641f50ae (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=2b529a82afc15a76185fca16ba7bcc8fbd82d15d
commit 2b529a82afc15a76185fca16ba7bcc8fbd82d15d
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Fri Oct 9 16:35:53 2015 -0400
Fix typo in bug-strcoll2 (Bug 18589)
Fix the copyright year and remove contributed by in the
bug-strcoll2 test. In addition add the correct dependency
on $(gen-locales) to ensure all the test locales are generated.
(cherry picked from commit facdd9ea29ab94aac2b188ec3cc41f8733d769e0)
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=c0769d60c1d107d9fb622e80c9d1110deb4fb3b4
commit c0769d60c1d107d9fb622e80c9d1110deb4fb3b4
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Thu Oct 8 16:54:30 2015 -0400
strcoll: Add bug-strcoll2 to testsuite (Bug 18589).
Adds bug-strcoll2 to the string tests, along with the
generation of required locales.
(cherry picked from commit 02018629a1397d03eccceacaf2ee1c50e3c4001c)
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=ed7f3a2e27e816488d5a8ef570052a5a2c3c515b
commit ed7f3a2e27e816488d5a8ef570052a5a2c3c515b
Author: Martin Sebor <msebor@gcc.gnu.org>
Date: Mon Sep 28 16:55:57 2015 -0400
Let 'make check subdirs=string' succeed even when it's invoked
immediately after glibc has been built and before 'make check'
(or after 'make clean').
(cherry picked from commit 60cf80f09d029257caedc0c8abe7e3e09c64e6c7)
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=6c84109cfa26f35c3dfed3acb97d347361bd5849
commit 6c84109cfa26f35c3dfed3acb97d347361bd5849
Author: Carlos O'Donell <carlos@systemhalted.org>
Date: Thu Oct 8 16:34:53 2015 -0400
strcoll: Remove incorrect STRDIFF-based optimization (Bug 18589).
The optimization introduced in commit
f13c2a8dff2329c6692a80176262ceaaf8a6f74e, causes regressions in
sorting for languages that have digraphs that change sort order, like
cs_CZ which sorts ch between h and i.
My analysis shows the fast-forwarding optimization in STRCOLL advances
through a digraph while possibly stopping in the middle which results
in a subsequent skipping of the digraph and incorrect sorting. The
optimization is incorrect as implemented and because of that I'm
removing it for 2.23, and I will also commit this fix for 2.22 where
it was originally introduced.
This patch reverts the optimization, introduces a new bug-strcoll2.c
regression test that tests both cs_CZ.UTF-8 and da_DK.ISO-8859-1 and
ensures they sort one digraph each correctly. The optimization can't be
applied without regressing this test.
Checked on x86_64, bug-strcoll2.c fails without this patch and passes
after. This will also get a fix on 2.22 which has the same bug.
(cherry picked from commit 87701a58e291bd7ac3b407d10a829dac52c9c16e)
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 37 +++++++++++++++++
NEWS | 2 +-
locale/C-collate.c | 4 +-
locale/categories.def | 1 -
locale/langinfo.h | 1 -
locale/localeinfo.h | 8 ----
locale/programs/ld-collate.c | 9 ----
string/Makefile | 15 ++++++-
string/bug-strcoll2.c | 92 ++++++++++++++++++++++++++++++++++++++++++
string/strcoll_l.c | 38 +-----------------
string/tst-strxfrm2.c | 6 ++-
wcsmbs/wcscoll_l.c | 1 -
12 files changed, 150 insertions(+), 64 deletions(-)
create mode 100644 string/bug-strcoll2.c
--
You are receiving this mail because:
You are on the CC list for the bug.