Re: uchar-c23: Speed up mbrtoc32 on Solaris 11.4

2024-05-25 Thread Collin Funk
On 5/25/24 3:15 AM, Bruno Haible wrote: > The Solaris 11.4 test run on GitHub is taking 4 hours, whereas it's only > 1 hour (at most) on the other platforms. > > In order to investigate this, I attached a time stamp to each output line: > > $make 2>&1 | gawk '{ print strftime("%H:%M:%S"), $0; f

euidaccess: Fix test failure on mingw

2024-05-25 Thread Bruno Haible
On mingw, I'm seeing test-euidaccess failures: ../../gltests/test-access.h:54: assertion 'errno == ENOTDIR' failed Stack trace: 0xcd2ea6c5 test_access ../../gltests/test-access.h:54 0xcd2ea6c5 main ../../gltests/test-euidaccess.c:37 ../../gltests/test-access.h:56: assertion 'errno

Re: mcel tests: Run the test in several locales

2024-05-25 Thread Bruno Haible
I did: > 2024-05-21 Bruno Haible > > mcel tests: Run the test in several locales. > * tests/test-mcel-1.sh: New file, based on tests/test-mbrtowc-1.sh. > * tests/test-mcel-2.sh: New file, based on tests/test-mbrtowc-2.sh. > * tests/test-mcel-3.sh: New file, based on test

mcel tests: Fix typo

2024-05-25 Thread Bruno Haible
Looking in detail at the mcel test, I noticed this obvious typo. The test was testing much fewer 3-character combinations than intended. This patch fixes it. It increases the execution time of test-mcel-[245].sh from ca. 4 seconds to 15-20 seconds (in a build with -O0), which should be acceptable.

uchar-c23: Speed up mbrtoc32 on Solaris 11.4

2024-05-25 Thread Bruno Haible
The Solaris 11.4 test run on GitHub is taking 4 hours, whereas it's only 1 hour (at most) on the other platforms. In order to investigate this, I attached a time stamp to each output line: $make 2>&1 | gawk '{ print strftime("%H:%M:%S"), $0; fflush(); }' | tee log2 $make check 2>&1 | gawk '{

tests: Before declaring a SKIP, test if there were ASSERT failures

2024-05-25 Thread Bruno Haible
Paul Eggert wrote: > * tests/test-linkat.c (main): Don’t lose the failure results of > earlier tests. Problem found by Coverity. The same problem exists on a few more tests. Fixed by the first patch below. The second patch is a simple refactoring. 2024-05-25 Bruno Haible tests: Ref

byteswap tests: Verify header can be used from C++.

2024-05-25 Thread Collin Funk
I've applied this patch adding a C++ test for the byteswap.h substitute. I guess there wasn't a need before since it only defined macros, but now it defines functions. CollinFrom 8658d279727e9e54d01d7ae38a9c5170893d3507 Mon Sep 17 00:00:00 2001 From: Collin Funk Date: Sat, 25 May 2024 01:20:01 -0