Converts C++ std::map to boost::python::dict

std::map_to_boost::python::dict.cpp

// Converts a C++ map to a python dict
template <class K, class V>
boost::python::dict toPythonDict(std::map<K, V> map) {
    typename std::map<K, V>::iterator iter;
	boost::python::dict dictionary;
	for (iter = map.begin(); iter != map.end(); ++iter) {
		dictionary[iter->first] = iter->second;
	}
	return dictionary;
}

I would rather put const std::map<K,V>& map as a parameter, that is a const ref. It won't change the code but can greatly help the compiler to optimize the code.

i think auto is better than typename std::map<K, V>::iterator iter;

addressing comment: iterator doesn't work with const argument here. Code works fine in snippet, though I'm passing by ref.

@AntonBrunyee, of course it is not working, because you need a const_iterator over iterator.

How to use this function?