On Mon, 28 Nov 2022 at 10:08, Jonathan Wakely <jwak...@redhat.com> wrote:

>
>
> On Mon, 28 Nov 2022 at 10:06, Jonathan Wakely <jwak...@redhat.com> wrote:
>
>>
>>
>> On Mon, 28 Nov 2022 at 06:02, François Dumont via Libstdc++ <
>> libstd...@gcc.gnu.org> wrote:
>>
>>> libstdc++: [_GLIBCXX_INLINE_VERSION] Adapt dg error messages
>>>
>>> libstdc++-v3/ChangeLog
>>>
>>>          * testsuite/20_util/bind/ref_neg.cc: Adapt dg-prune-output
>>> message.
>>>          * testsuite/20_util/function/cons/70692.cc: Adapt dg-error
>>> message.
>>>
>>> Ok to commit ?
>>>
>>>
>> OK, thanks.
>>
>>
>>
> Actually wait, can you test this instead?
>
> --- a/libstdc++-v3/testsuite/lib/prune.exp
> +++ b/libstdc++-v3/testsuite/lib/prune.exp
> @@ -37,6 +37,8 @@ proc libstdc++-dg-prune { system text } {
>       return "::unsupported::hosted C++ headers not supported"
>     }
>
> +    regsub -all "std::__8::" $text "std::" text
> +
>     # Ignore caret diagnostics. Unfortunately dejaGNU trims leading
>     # spaces, so one cannot rely on them being present.
>     regsub -all "(^|\n)\[^\n\]+\n *\\^\n" $text "\n" text
>
> This should mean we can stop needing to make these changes to every test,
> and just write the tests naturally.
>

That only helps for dg-prune-output but we still need to (__8::)? for
dg-error.

Please push your change to 20_util/function/cons/70692.cc but not the
change to 20_util/bind/ref_neg.cc (the latter will get fixed after I pushed
the prune.expo change).

Reply via email to