STLPort multimap problem or mine ?

[Yves M]

It is probably my problem, but I don't see what's wrong here :/

I'm trying to delete a specifc element in a multimap based on the key and value since the multimap can store multiple entries with the same key.

#include <map>
using namespace std;

struct eqstr
{
  bool operator()(const char* s1, const char* s2) const
  {
    return strcmp(s1, s2) == 0;
  }
};

typedef multimap<const char*, int, eqstr> map_type;

void eraseElem(const map_type& Map, const char* str, const int key)
{
  pair<map_type::const_iterator, map_type::const_iterator> p = Map.equal_range(str);
  map_type::const_iterator it = p.first;
  while ((it != p.second) && ((*it).second != key)) {
    ++it;
  }
  if (it != p.second) {
    Map.erase(it); // <- this line doesn't compile
  }
}

int main()
{
  map_type M;
  M.insert(map_type::value_type("H", 1));
  M.insert(map_type::value_type("H", 2));
  M.insert(map_type::value_type("C", 12));
  M.insert(map_type::value_type("C", 13));
  M.insert(map_type::value_type("O", 16));
  M.insert(map_type::value_type("O", 17));
  M.insert(map_type::value_type("O", 18));
  M.insert(map_type::value_type("I", 127));

  eraseElem(M, "O", 18);
  return 0;
}

The error I get is :
'erase' : 3 overloads have no legal conversion for 'this' pointer

I have tried it both with const_iterator and iterator. I can't really see what is wrong, since the only condition that is documented is that it is a dereferenceable iterator in Map. Or is this iterator not in Map ?

[Paul McKenzie]

Remove the "const" from your function prototype.

void eraseElem(map_type& Map, const char* str, const int key)

Regards,

Paul McKenzie

[Yves M]

Thanks a lot

Ups that was a silly one