The code below (apologies for the length of it, but it's the only way I found to reproduce the error) fails to compile using VS2013. It seems to have something to do with expanding an empty parameter pack. However, when I remove any of the three lines in main(), the compilation error goes away. This leads me to believe it is probably a bug - especially also because IDEone handles it fine :P
The compilation errors, line 28 is the one with using tuple_t = std::tuple< typename remove_const_and_reference< A >::type... >;
:
(28): error C3548: 'A' : parameter pack cannot be used in this context
(19): see reference to function template instantiation 'void A::CallCore<Ret,>(Ret (__cdecl *)(void))' being compiled
with
[
Ret=Foo *
]
(50): see reference to function template instantiation 'void A::Call<T*,>(Ret (__cdecl *)(void))' being compiled
with
[
T=Foo,Ret=Foo *
]
(77): see reference to function template instantiation 'void B::RegisterConstructor<Foo,>(void)' being compiled
(28): error C2976: 'remove_const_and_reference' : too few template arguments
(28): error C2955: 'remove_const_and_reference' : use of class template requires template argument list
(28): error C2039: 'type' : is not a member of 'remove_const_and_reference'
(28): error C2146: syntax error : missing ',' before identifier 'type'
(28): error C3546: '...' : there are no parameter packs available to expand
(28): error C2065: 'type' : undeclared identifier
(28): error C3544: '_Types': parameter pack expects a type template argument
The question: I'm looking for a workaround and some insight into why there could be a problem?
Code:
#include <tuple>
#include <type_traits>
template< class T, class U = typename std::remove_cv< typename std::remove_reference< T >::type >::type >
struct remove_const_and_reference : remove_const_and_reference< U >
{
};
template< class T >
struct remove_const_and_reference< T, T >
{
typedef T type;
};
class A
{
public:
template< class Ret, class... Args >
void Call( Ret f( Args... ) ) { CallCore( f ); }
template< class Ret, class Obj, class... Args >
void Call( Ret( Obj::*f )( Args...) ) { CallCore( f ); }
private:
template< class Ret, class... Args >
void CallCore( Ret f( Args... ) )
{
using tuple_t = std::tuple< typename remove_const_and_reference< Args >::type... >;
}
template< class Ret, class Obj, class... Args >
void CallCore( Ret( Obj::*f )( Args...) )
{
using tuple_t = std::tuple< typename remove_const_and_reference< Args >::type... >;
}
};
template< class T, class... Args >
T* New( Args&&... args )
{
return new T( std::forward< Args >( args )... );
}
class B
{
public:
template< class T, class... Args >
void RegisterConstructor()
{
a.Call< T* >( New< T, Args... > );
}
template< class Fun >
void Register( Fun f )
{
a.Call( f );
}
private:
A a;
};
struct Foo
{
Foo(){}
~Foo(){}
void FunA( int ){}
};
void FunB( int ){}
int main()
{
B().Register( FunB );
B().Register( &Foo::FunA );
B().RegisterConstructor< Foo >();
}
To make it working in VS2013:
1.A
names your class, you should not use template <class A>
within class A
's scope. Rename your parameter pack into let's say Args
:
template< class Ret, class Obj, class... Args > // << Use Args name, not A !
void CallCore( Ret( Obj::*f )( Args...) )
{
using tuple_t = std::tuple< typename remove_const_and_reference< Args >::type... >;
}
2.I don't get you struct (why it inherits itself?), change it to:
template <typename T>
struct remove_const_and_reference
{
using type = typename std::remove_cv< typename std::remove_reference< T >::type >::type;
};