[PATCH][_GLIBCXX_INLINE_VERSION] Adapt dg error messages
François Dumont
frs.dumont@gmail.com
Mon Nov 28 17:59:27 GMT 2022
On 28/11/22 14:39, Jonathan Wakely wrote:
>
>
> On Mon, 28 Nov 2022 at 10:08, Jonathan Wakely <jwakely@redhat.com> wrote:
>
>
>
> On Mon, 28 Nov 2022 at 10:06, Jonathan Wakely <jwakely@redhat.com>
> wrote:
>
>
>
> On Mon, 28 Nov 2022 at 06:02, François Dumont via Libstdc++
> <libstdc++@gcc.gnu.org <mailto:libstdc%2B%2B@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).
>
>
Done as requested and I confirm that prune.exp enhancement fixed
20_util/bind/ref_neg.cc.
Thanks
More information about the Libstdc++
mailing list