[PATCH] D34331: func.wrap.func.con: Unset function before destroying anything

[Volodymyr Sapsai via Phabricator via cfe-commits]

vsapsai added a comment.

It is agreed that it is untenable to support reentrancy for `std::function` assignment operators and I'm not trying to achieve that. Here we restrict reentrancy only to `std::function::operator=(nullptr_t)`.

@mclow.lists where should the tests go if the standard specifies the functionality as implementation-defined?

> Except where explicitly specified in this standard, it is implementation-defined which functions in the Standard C++ library may be recursively reentered.

[reentrancy] p17.6.5.8


Repository:
  rCXX libc++

https://reviews.llvm.org/D34331