Исправление XMs, (текущая версия) :
Как уже упомянули — не thread safe, если только ты не озаботишься этим самостоятельно. Непонятно, зачем делать так, если можно не жертвовать понятностью кода, создав указатель (желательно умный) на уже существующий экземпляр класса либо ссылку. В третьих — так ты шаришь не полностью, часть контейнера оказывается не шаренной, а скопированной. Ты знаешь, как устроен внутри std::map и чем placement new отличается от memcpy?
// А вообще выше верно анон написал — лучше дёрнуть конструктор копирования вместо placement new
Исходная версия XMs, :
Как уже упомянули — не thread safe, если только ты не озаботишься этим самостоятельно. Непонятно, зачем делать так, если можно не жертвовать понятностью кода, создав указатель (желательно умный) на уже существующий экземпляр класса либо ссылку. В третьих — так ты шаришь не полностью, часть контейнера оказывается не шаренной, а скопированной. Ты знаешь, как устроен внутри std::map и чем placement new отличается от memcpy?