what is wrong with this generic print function ?

#include <iostream>
#include <vector>

using namespace std;

template<typename T>
void printvec(vector<T> var)
{
   for (vector<T>::iterator i=var.begin(); i!= var.end(); i++)
   {
     cout<<" value is "<<*i<<endl;
   }
}

int main()
{
   vector<int> vec;

   vec.push_back(10);
   vec.push_back(20);
   vec.push_back(30);
   vec.push_back(40);

   printvec(vec);
}

+++++compiler error is++++

vector.cc: In function ?void printvec(std::vector<T,
std::allocator<_CharT> >)?:
vector.cc:10: error: expected `;' before ?i?
vector.cc:10: error: ?i? was not declared in this scope
vector.cc: In function ?void printvec(std::vector<T,
std::allocator<_CharT> >) [with T = int]?:
vector.cc:25: instantiated from here
vector.cc:10: error: dependent-name ?std::vector::iterator? is parsed
as a non-type, but instantiation yields a type
vector.cc:10: note: say ?typename std::vector::iterator? if a type is
meant

{ Short answer: use "typename" in front of "vector<T>::iterator". -mod/sk }

--
      [ See http://www.gotw.ca/resources/clcm.htm for info about ]
      [ comp.lang.c++.moderated. First time posters: Do this! ]