[libcxx-commits] [PATCH] D47344: LWG 2843 "Unclear behavior of std::pmr::memory_resource::do_allocate()"
Eric Fiselier via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Mar 6 13:47:36 PST 2019
EricWF added inline comments.
Comment at: src/experimental/memory_resource.cpp:29
- ~__new_delete_memory_resource_imp() = default;
> EricWF wrote:
> > Why are you removing this?
> Just removing unnecessary cruft. Same reason I added `override` (and removed `virtual`) on the other methods, and removed `protected:`, and removed the `__` sigilling on function parameters in this .cpp file.
> I had done a lot more random cleanup originally, and then you said "unrelated, please revert" so I did, but I assume I just kept all the cleanups in this one class since I was touching this range of lines no matter what.
I strongly prefer seeing explicit declarations of non-trivial or virtual destructors.
CHANGES SINCE LAST ACTION
More information about the libcxx-commits