On Sat, 18 Apr 2020 at 03:35, kamlesh kumar via Libstdc++ <[email protected]> wrote: > > On Fri, Apr 17, 2020, 10:59 PM kamlesh kumar <[email protected]> > wrote: > > > Fixes all this. > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92156 > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91630 > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90415 > > > > On Fri, Apr 17, 2020 at 10:45 PM kamlesh kumar <[email protected]> > > wrote: > > > > > > This patch corrects the requirement of 4,5 and 6th constructor > > > As per https://en.cppreference.com/w/cpp/utility/any/any.
The patch looks correct to me. We have some old cruft there, like the overload your patch removes, it was there to support copy-only types, but LWG issues axed that bit. This constructor indeed should not check is_constructible, because it'll end up instantiating this constructor itself, and compute its constraints, and instantiate itself. The in_place constructor doesn't have that problem, because it won't instantiate itself.
