[RESOLVED] What's wrong with this STL code

Compiled in 2008, gives error C2275, illegal use of this type as an expression in 2012

Code:

---

template
<
        class tSTLContainer                        // The type of STL container to copy.
>
void STLCopy (tSTLContainer& outDest, const tSTLContainer& inSource)
{
        copy(inSource.begin(), inSource.end(), insert_iterator<tSTLContainer>(outDest, outDest.begin()));
}

---

Never mind. Apparently you need to #include <iterator> now.

I don't have 2012 in front of me, but maybe you forgot to include <iterator>?

Here is a full code example

Code:

---

#include <algorithm>
#include <iterator>
#include <vector>
#include <set>
#include <map>

using namespace std;
template <class tSTLContainer>
void STLCopy (tSTLContainer& outDest, const tSTLContainer& inSource)
{
    std::copy(inSource.begin(), inSource.end(), std::insert_iterator<tSTLContainer>(outDest, outDest.begin()));
}

int main()
{
  std::map<int, int> InMap, OutMap;
  STLCopy(OutMap, InMap);
  std::set<int> InSet, OutSet;
  STLCopy(OutSet, InSet);
  std::vector<int> InVect, OutVect;
  STLCopy(OutVect, InVect);
  return 0;
}

---

Regards,

Paul McKenzie

Already figured that out, but thanks, you're right.

Quote:

---

Originally Posted by GCDEF

Already figured that out, but thanks, you're right.

---

The <iterator> header is a real sneaky one in that so many compilers accept code without it being included.

Regards,

Paul McKenzie

Quote:

---

Originally Posted by Paul McKenzie

The <iterator> header is a real sneaky one in that so many compilers accept code without it being included.

Regards,

Paul McKenzie

---

<functional> is the same way. Not required in 2008 but necessary in 2012. Got a clean build once I figured those two out.