Search code examples
c++c++11templatescompiler-errorsvariadic-templates

VS2013 Variadic template compilation error


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 >();
}

Solution

  • 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;
    };