
On 8/23/06, Minkoo Seo <minkoo.seo@gmail.com> wrote:
for_each(m.begin(), m.end(), bind(&map<string, int>::insert, &m2, bind(&make_pair, bind(&map<string, int>::value_type::first, _1), bind(&map<string, int>::value_type::second, _1))));
Unfortunately, the above code fails to compile and I have no idea why it fails. I'm using g++ and the errors are as follows:
First off, just thought I would mention that for an example such as this, regular boost::bind would probably suffice. You don't need lambda. Second of all, I don't know which insert function you are trying to call, but you should be able to write this code instead, which is much simpler: for_each(m.begin(), m.end(), bind(&map<string, int>::insert, m2, _1)); Finally, even that fails to compile! I believe the problem is that since insert is an overloaded function, boost cannot figure out which one to call. So you help it out by type-casting explicitly, like this: typedef pair<map<string, int>::iterator, bool> (map<string, int>::* InsertFunc)(const map<string, int>::value_type&); for_each(m.begin(), m.end(), bind(static_cast<InsertFunc>(&map<string, int>::insert), m2, _1)); It's ugly, but it works (actually, I tried with lambda and it still didn't work, but with regular boost::bind it does). Now after going to ALL that work just so you can use for_each, don't you just want to forget it and use iterators? for( map<string, int>::const_iterator it = m.begin(); it != m.end(); ++it ) m2.insert(*it); Phillip Hellewell