[PATCH] PR libstdc++/85184 remove debug assertions from std::variant

[Ville Voutilainen]

On Wed, 3 Apr 2019 at 13:04, Ville Voutilainen
<ville.voutilainen@gmail.com> wrote:
>
> On Wed, 3 Apr 2019 at 12:48, Jonathan Wakely <jwakely@redhat.com> wrote:
> >
> > The __glibcxx_assert macro should be used to check preconditions that
> > users must meet, not to check postconditions that the implementation
> > must meet. We have tests to verify std::variant meets its
> > postconditions, users shouldn't pay for those checks at runtime.
> >
> >         PR libstdc++/85184
> >         * include/std/variant (_Copy_assign_base, _Move_assign_base, variant):
> >         Remove assertions.
> >         (variant::emplace<_Tp>): Remove result of emplace<N> directly.
> >
> > Tested powerpc64le-linux, committed to trunk.
>
> The latter changelog entry should say "Return resulf of emplace<N>
> directly", not "Remove result...".

And yeah, *result*, dunno how that f came into existence instead of a
t. Read it with the right font so that a ligature
hides it. :P