| Message ID | CAFULd4b--iZpW1qRZAveoaCwKFBrcRRhNRAnZ8bay=pWT5VEHQ@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | libstdc++: Fix 22_locale/numpunct/members/char/3.cc execution test | expand |
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.
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);
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.