[PATCH] D11111: [libcxx] Fix PR23589: std::function doesn't recognize null pointer to varargs function.
Marshall Clow
mclow.lists at gmail.com
Tue Jul 21 08:11:45 PDT 2015
mclow.lists added a comment.
This looks like a nice simplification, and the test is great.
However, I have some concerns about where the pieces live - see inline comments.
================
Comment at: include/__functional_03:207
@@ -206,3 +206,1 @@
-template<class _Fp> class _LIBCPP_TYPE_VIS_ONLY function; // undefined
-
namespace __function
----------------
It's not clear to me that removing this declaration is correct. There are specializations of `function` in this file.
================
Comment at: include/__functional_03:761
@@ -771,3 +760,3 @@
{
- if (__not_null(__f))
+ if (__function::__not_null(__f))
{
----------------
Where is `__functional_03` getting the definition of `__not_null` from?
It doesn't include any other headers.
http://reviews.llvm.org/D11111
More information about the cfe-commits
mailing list