Ох уж этот std::make_shared…

Честно говоря, я очень долгое время думал, что основное назначение std::make_... — возможность не писать тип в контейнере два раза.

• new Bar;
• foo();
• конструктор std::shared_ptr.

Готовил длинную гневную тираду, потом посмотрел в доки — а оно похоже так и есть. Нет слов, одни выражения...

к сожалению, без профилирования нельзя четко сказать: бесполезный и вредный.

• ситуация 1: я шарю int и часто к нему обращаюсь. Очень хорошо, что он в одном блоке с control_block и наплевать, что 4байта памяти будут висеть на слабых ссылках
• ситуация 2: я шарю вектор на 10к элементов и обращаюсь к нему редко. Здесь верно все то, о чем в статье написано

Так что в качестве вывода можно написать, что std::make_shared лучше подходит для малых объектов(до 64 — sizeof(control_block)) и частых обращений. И приводет к подвисанию памяти на больших объектах.

По поводу custom deleter — "не поддерживает" и "не возможно сказать устновлен ли" — это разные характеристики, не правда ли?

По поводу custom new и custom delete — да, но так ли они нужны, когда проще просто залинковать какойнибудь jmalloc?

#include <iostream>
#include <chrono>
#include <list>

template<class T, class F>
auto test(F&& create)
{
  std::list<std::shared_ptr<T>> set;
  auto max = std::numeric_limits<size_t>::max();
  for (size_t count = 0; count < max - 1; ++count)
  {
    try
    {
      set.emplace_back(create(count));
    }
    catch (const std::bad_alloc&)
    {
      return count;
    }
  }

  return max;
}

int main(int argc, char *argv[])
{
  using namespace std::chrono;
  using type = int;
  if (argc == 1)
  {
    auto start = steady_clock::now();
    auto val = test<type>([](auto val) { return std::make_shared<type>(val); });
    auto end = steady_clock::now();
    std::cout << "make_shared | count: " << val << ", time: " << duration_cast<milliseconds>(end - start).count() << "msc\n";
  }
  else
  {
    auto start = steady_clock::now();
    auto val = test<type>([](auto val) { return new type(val); });
    auto end = steady_clock::now();
    std::cout << "new | count: " << val << ", time: " << duration_cast<milliseconds>(end - start).count() << "msc\n";
  }
}