Connector cities

Reads and insert in database all cities of a given osm file
Why don't you remove osm_id from these struct, remove these mutable things, and build std::map<osm_id, OSMStruct>?