map.insert - что я делаю не так?

Что я делаю не так? Нужно просто написать функцию, которая значения из map переводит в set

#include <iostream>
#include <vector>
#include <set>
#include <string>
#include <map>
std::set<std::string> BuildMapValuesSet (const std::map<int, std::string>& mp){
  std::set<std::string> s;
  for(auto item : mp)
    {
        s.insert(item.first);
    }
  return s;
}

int main(){
  std::set<std::string> values = BuildMapValuesSet({
      {1, "odd"},
      {2, "even"},
      {3, "odd"},
      {4, "even"},
      {5, "odd"}
  });
  for (const std::string& value : values) {
    std::cout << value;
  }
}