libstdc++: Fix 22_locale/numpunct/members/char/3.cc execution test

Message ID CAFULd4b--iZpW1qRZAveoaCwKFBrcRRhNRAnZ8bay=pWT5VEHQ@mail.gmail.com
State Committed
Headers
Series libstdc++: Fix 22_locale/numpunct/members/char/3.cc execution test |

Commit Message

Uros Bizjak Jan. 14, 2022, 11:41 a.m. UTC
  The test fails on Fedora 33+ because nl_NL locale got thousands
separator defined.  Use one of ar_SA, bg_BG, bs_BA, pt_PT
or plain C locale instead.

2022-01-14  Uroš Bizjak  <ubizjak@gmail.com>

libstdc++-v3/ChangeLog:

       * testsuite/22_locale/numpunct/members/char/3.cc (test02):
       Use pt_PT locale instead of nl_NL.

Tested on x86_64-linux-gnu (Fedora 33).

OK for master?

Uros.
  

Comments

Jonathan Wakely Jan. 14, 2022, 12:13 p.m. UTC | #1
On Fri, 14 Jan 2022 at 11:41, Uros Bizjak via Libstdc++ <
libstdc++@gcc.gnu.org> wrote:

> The test fails on Fedora 33+ because nl_NL locale got thousands
> separator defined.  Use one of ar_SA, bg_BG, bs_BA, pt_PT
> or plain C locale instead.
>
> 2022-01-14  Uroš Bizjak  <ubizjak@gmail.com>
>
> libstdc++-v3/ChangeLog:
>
>        * testsuite/22_locale/numpunct/members/char/3.cc (test02):
>        Use pt_PT locale instead of nl_NL.
>
> Tested on x86_64-linux-gnu (Fedora 33).
>
> OK for master?
>
>
Please also change the dg-require-namedlocale directive on the first line.

OK with that change, thanks.
  

Patch

diff --git a/libstdc++-v3/testsuite/22_locale/numpunct/members/char/3.cc b/libstdc++-v3/testsuite/22_locale/numpunct/members/char/3.cc
index 6e4959860cd..4cf4b24407d 100644
--- a/libstdc++-v3/testsuite/22_locale/numpunct/members/char/3.cc
+++ b/libstdc++-v3/testsuite/22_locale/numpunct/members/char/3.cc
@@ -28,8 +28,8 @@  void test02()
 {
   using namespace std;
 
-  // nl_NL chosen because it has no thousands separator (at this time).
-  locale loc_it = locale(ISO_8859(15,nl_NL));
+  // pt_PT chosen because it has no thousands separator (at this time).
+  locale loc_it = locale(ISO_8859(15,pt_PT));
 
   const numpunct<char>& nump_it = use_facet<numpunct<char> >(loc_it);